Fixing Json Parsing Issue #2550
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
As discussed on slack, merged the PR. |
Madanagopal19
added a commit
that referenced
this pull request
Jun 4, 2024
* kubevirt vendor updates for schedops (#2420) Signed-off-by: Rohit-PX <rkulkarni@purestorage.com> Co-authored-by: Rohit-PX <rkulkarni@purestorage.com> * PB-6789:Create an app/vm in specific namespace (#2410) * Fix regex spelling error (#2429) * Fix restore NS name (#2430) * Ingore check for purefile (#2426) * Ingore check for purefile * [TC Automation] PTX-21440 Add test case VolAttachFAPxRestart (#2361) *[TC Automation] PTX-21440 Add test case VolAttachFAPxRestart,VolAttachSameFAPxRestart * Adding check to ignore Volume attaches in case of pure file volume (#2432) * Adding cloud create and s3 bucket deletion function for cloudsnaps (#2406) * Adding cloud create and s3 bucket deletion function for cloudsnaps * Adding bucket cleanup as part of cloudsnap tests --------- Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * Adding ZRS spec for aks (#2393) * Added method for creating volume snapshot class with the given provisioner (#2418) * PB-6789:Create an app/vm in specific namespace * Added snapclass creation * PB-5946: Added logic for PVC cleanup validation (#2282) * Added logic for PVC cleanup validation * Ptx 23576 VolResizeAllVolumes in parallel (#2433) * Trigger vol resize on all volumes at once * PTX-23652: be resilient to failures of GetVolumes() call (#2425) * PTX-23652: be resilient to failures of GetVolumes() call Signed-off-by: pnookala-px <pnookala@purestorage.com> * move total computation to before settting migration Signed-off-by: pnookala-px <pnookala@purestorage.com> * Retry GetVolumes() call Signed-off-by: pnookala-px <pnookala@purestorage.com> --------- Signed-off-by: pnookala-px <pnookala@purestorage.com> * Create and attach 250 + FADA volumes in Longevity or scale testing automation (#2421) Signed-off-by: Smit Jain <smjain@purestorage.com> * Fixing volume provisoner in case of back up test (#2437) Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * Add a new test case for bootstrapping with local disks (#2428) * Added a new testcase for PTX-22628 * Change to MoveDisks * skip the root disk for detaching * cosmetic changes * Different logic for move disks * Updating node.go * Changes with Detach and AttachDisk * Added print statements * typo * Try with sleep * test detach attachdisk * try with RemoveDevice * moving the comment * testing objects * change to moref.value * change to use the existing finder object * debug statement * newer logic * different way to fetch datastore object * debug statement * old logic * try with replace file op * remove replace * cleanup code * wait for PX to be online * Added logic to remove disks for nonpxnodes * Update node.go * Added log statement for disk operations * temp: commenting out debug statement * ignore master nodes for targetVM * graceful movement of disks, add PXstop to srcVM * add log state for failure * update registry * refreshing driverendpoints * disabling PX on the stopped node * adding another refresh node registry * adding a log statement for debug * update the randNonPxNode object * Added a log statement for successful check of node ID * Test log statement cleanup * move the test to misc_test * correct the import sections * addressed review comments * Added pxbackupversion check to verify api call (#2434) * [PTX-23625] Fix TriggerRebootManyNodes (#2442) * Test automation for VolHAIncreaseAllVolumes (#2441) * Test automation for VolHAIncreaseAllVolumes * Anthos upgrade is failing in anthos reconcile preflight check (#2443) Signed-off-by: Smit Jain <smjain@purestorage.com> * fixing MultiVolumeMountsForSharedV4 for chos-level (#2446) Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * [PWX-37026] Adding FBDAMultiTenancyBasicTest (#2419) * [PWX-37026] Adding FBDAMultiTenancyBasicTest Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * [PWX-37026] Updated libopenstorage/openstorage dependancy Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * [PWX-37026] Added FBDAMultiTenancyBasicTest Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * [PWX-37026] Addressed review comments Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> --------- Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * Add test case to bring down network interface of FA Controller (#2455) * Add test case to bring down network interface of FA Controller * Adding senarios for adding storage node and stoageless nodes for clou… (#2449) * Adding senarios for adding storage node and stoageless nodes for cloud platforms * Deleting node scaling reduntant code --------- Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@purestorage.com> * Adding test for new storkctl flow (#2431) * Ptx 23763 Get FB Cluster Details (#2457) * Add test case to bring down network interface of FB * Calculate node upgrade time for Anthos parallel upgrade (#2387) * Calculate node upgrade time for parallel upgrade * Updating vendoring for latest sched-ops Signed-off-by: Smit Jain <smjain@purestorage.com> --------- Signed-off-by: Smit Jain <smjain@purestorage.com> * Fixing the user name length because it is used in namespace creation and exceeding 63 chars (#2459) * Automation scripts for iscsi port drop scenarios [ PTX-23831, PTX-23832,PTX-23833,PTX-23818] (#2460) * Automation scripts for iscsi port drop scenarios * [PWX-33177] Skip pool expansions when app has only FBDA volumes. (#2456) Since FBDA volumes do not get created on PX pool, there is no use of this validation. Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * PB-6777: Add Backup driver with WaitForBackupPartialCompletion method and its portworx implementation (#2448) * Fix retry for GetPxBackupVersionString (#2471) * Fix for NVME parsing (#2469) * Updating string split logic to split volume serial, extracted from the mount path * New Test case addition (#2466) * New Test case addition deletes Volume Delete on FA Backend * Fix nfs loc validation and pvc cleanup (#2474) * Adding test to recycle all storage driver nodes and validate drive set in cloud platforms (#2473) * adding debug logs for usable address * Adding test to recycle all nodes * Adding test to recycle all nodes --------- Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * [TC Improvement] Take px namespace where it is installed (#2476) * Temporary removeal of pvc cleanup validation (#2478) * Added retry for backup location validation (#2479) * Removed data validation after restore due to PB-6271 (#2480) * [PTX-23856] Increase Timeout In ValidateStorageCluster (#2465) * Ptx 23988 (#2486) function to check and list all the multipath devices present in the node * PB-6917: Add `CUSTOM_REGISTRY` and `CUSTOM_REPO` to support custom registry deployment (#2477) * PB-6917: Add fix to use docker.io in case of cloud * PB-6917: Add fix to use docker.io in case of cloud * Change implementation to `CUSTOM_REGISTRY ` and `CUSTOM_REPO ` * PB-6917: Add fix to use docker.io in case of cloud * Change implementation to `CUSTOM_REGISTRY ` and `CUSTOM_REPO ` * Minor fix * Minor fix * Adding test to increase max nodes per zone after scale up (#2484) Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * Ptx 23988 (#2490) * functions to fetch details on CLoud drives , multipaths , and skip Adddisk if not supported * [PWX-37028] Added FBDAMultiTenancyUpdatePureNFSEnpoint (#2485) * [PWX-37028] Added FBDAMultiTenancyUpdatePureNFSEnpoint * [PWX-37028] Addressed review comments Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> --------- Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> * Run torpedo as a pod in anthos user cluster (#2491) Signed-off-by: Smit Jain <smjain@purestorage.com> * PTX-24028 Vendor latest libopenstorage/operator package (#2489) Signed-off-by: nikolaypopov <nikolay.popov86@gmail.com> * Kubevirt specs for datavoltemplate , win22 , large datadisk (#2376) * Kubevirt specs for datavoltemplate , win22 , large datadisk * Renamed files * fix nit in vm.yaml to accept .vmID * fix small nit * rename large disk app * remove long names * New test case to Expand Multiple pool when FADA volumes are getting C… (#2495) * New test case to Expand Multiple pool when FADA volumes are getting Created * [PWX-37028] Skip volume validations. (#2497) * [TC Automation] kubevirt test cases (#2483) * Fbda topology creates (#2496) * fbda with top test1 Signed-off-by: pnookala-px <pnookala@purestorage.com> * fbda with top test1 Signed-off-by: pnookala-px <pnookala@purestorage.com> * fbda with top test1 Signed-off-by: pnookala-px <pnookala@purestorage.com> * fbda with top test2 Signed-off-by: pnookala-px <pnookala@purestorage.com> * sleep 30 seconds after deleting pvc Signed-off-by: pnookala-px <pnookala@purestorage.com> * create sc without top Signed-off-by: pnookala-px <pnookala@purestorage.com> * create sc without top Signed-off-by: pnookala-px <pnookala@purestorage.com> * fix spelling Signed-off-by: pnookala-px <pnookala@purestorage.com> * wait 3 mins only Signed-off-by: pnookala-px <pnookala@purestorage.com> * Refactored common code --------- Signed-off-by: pnookala-px <pnookala@purestorage.com> Co-authored-by: Vinayak Shinde <vinayakshnd@gmail.com> * fixing debian VM spec to have internal mount (#2507) * adding retry with timeout for PDB validation (#2500) * [TC Automation] Add test case RestartPXAndCheckIfVmBindMount (#2510) * add test case RestartPXAndCheckIfVmBindMount * increasing cpu cores for bonnie app (#2488) Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> * Vendored the latest px-backup api for 2.7.1 (#2506) * Power off and power on all worker nodes (#2436) Signed-off-by: smarakala_pure <smarakala@purestorage.com> Co-authored-by: smarakala_pure <smarakala@purestorage.com> * [PTX-24172] Fix validateAutoFsTrim (#2512) * Detach clouddrives while IO inprogress and verify pool can be back after maintenance cycle (#2408) * Adding detach drives to longevity Signed-off-by: smarakala_pure <smarakala@purestorage.com> Co-authored-by: smarakala_pure <smarakala@purestorage.com> * minor fix for update node id function (#2515) Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@purestorage.com> * [PTX-24076] Add Ginkgo Support To Deployment Script (#2509) * [PTX-24076] Add yq Install Command To Dockerfile * [PTX-24076] Add Ginkgo Support To Deployment Script * [PTX-23863] Fix SetupProxyServer (#2524) * PWX-36878: DMthin OCP VSphere pool expand test update (#2503) * DMthin OCP VSphere pool expand test update Signed-off-by: Priyanshu Pandey <ppandey@purestorage.com> --------- Signed-off-by: Priyanshu Pandey <ppandey@purestorage.com> * Adding cloudsnap delete event to longevity (#2444) * Adding cloudsnap delete event to longevity Signed-off-by: smarakala_pure <smarakala@purestorage.com> * Adding params for px-security (#2527) * Fixing OCP zones (#2516) * Fixing OCP zones * Fixing ASG for openshift --------- Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@purestorage.com> * Removed aks node driver (#2533) * PTX-23958: Vendoring latest operator (#2531) * vendoring latest operator * vendoring after rke2 change * even newer operator code * PTX-24002 Create Volume clone for FADA volume and verify new volume is created in the backend (#2501) * PTX-24002 * PTX-24002 * PTX-24002 changes made according to comments * changes done to exit the loop if clone volume found in FA * changes done according to comments * minor nitpick changes * nitpick * New Libraries for FA RestAPIs 2.x (#2543) New Libraries for FA RestAPIs 2.x * Fixing Json Parsing Issue (#2550) Fixing Json Parsing Issue * Added mysql & postgres spec with restricted PSA (#2521) * Waiting for atleast 2 mongo pods to come up after scaling down to 0 and scaling backup to 3 again (#2552) * name change validation * latest pds api changes * added bronze acc * Fixed names in status * fixing manifest issue --------- Signed-off-by: Rohit-PX <rkulkarni@purestorage.com> Signed-off-by: pnookala-px <pnookala@purestorage.com> Signed-off-by: Smit Jain <smjain@purestorage.com> Signed-off-by: Vinayak Shinde <vinayakshnd@gmail.com> Signed-off-by: nikolaypopov <nikolay.popov86@gmail.com> Signed-off-by: smarakala_pure <smarakala@purestorage.com> Signed-off-by: Priyanshu Pandey <ppandey@purestorage.com> Co-authored-by: Rohit-PX <32072909+Rohit-PX@users.noreply.github.com> Co-authored-by: Rohit-PX <rkulkarni@purestorage.com> Co-authored-by: snigdha-px <110967459+snigdha-px@users.noreply.github.com> Co-authored-by: suhas <122518640+spai-px@users.noreply.github.com> Co-authored-by: Kshithij Iyer <81960124+kshithijiyer-px@users.noreply.github.com> Co-authored-by: vprabhakar-px <120549478+vprabhakar-px@users.noreply.github.com> Co-authored-by: Leela Srinivas <83946232+lsrinivas-pure@users.noreply.github.com> Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@.purestorage.com> Co-authored-by: Adil Rahman <123445738+ak-px@users.noreply.github.com> Co-authored-by: sabrarhussaini <105486766+sabrarhussaini@users.noreply.github.com> Co-authored-by: pnookala-px <127919775+pnookala-px@users.noreply.github.com> Co-authored-by: jainsmit <smjain@purestorage.com> Co-authored-by: vumesh-px <125877317+vumesh-px@users.noreply.github.com> Co-authored-by: kphalgun-px <122591456+kphalgun-px@users.noreply.github.com> Co-authored-by: Vinayak Shinde <vinayakshnd@gmail.com> Co-authored-by: Leela Srinivas Siripurapu <lsrinivas@purestorage.com> Co-authored-by: Geet-PX <115655007+gejain@users.noreply.github.com> Co-authored-by: Mithun Koppal <114994232+mkoppal-px@users.noreply.github.com> Co-authored-by: sgajawada-px <116876049+sgajawada-px@users.noreply.github.com> Co-authored-by: Nikolay Popov <7504816+nikolaypopov@users.noreply.github.com> Co-authored-by: px-kannavarapu <147571353+px-kannavarapu@users.noreply.github.com> Co-authored-by: Dhruv Bhatnagar <117347853+dbhatnagar-px@users.noreply.github.com> Co-authored-by: Swarupa Vijaykumar <123443458+svijaykumar-px@users.noreply.github.com> Co-authored-by: Santhosh Marakala <75639214+Santhoshairody@users.noreply.github.com> Co-authored-by: smarakala_pure <smarakala@purestorage.com> Co-authored-by: Priyanshu Pandey <ppandey@purestorage.com> Co-authored-by: sayalasomayajula-px <162689439+sayalasomayajula-px@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixing Parsing issues with Rest APIS
2024-05-28 23:29:18 +0530:[INFO] [flasharray.(*Client).NewRequest:#174] - Base URL [https://10.9.1.152/api/2.4/volumes?destroyed=true]
2024-05-28 23:29:18 +0530:[INFO] [flasharray.(*Client).NewRequest:#191] - Adding auth token [*********75d58d]
2024-05-28 23:29:18 +0530:[INFO] [flasharray.(*Client).Do:#124] -
&{200 OK 200 HTTP/1.1 1 1 map[Connection:[keep-alive] Content-Length:[694] Content-Type:[application/json] Date:[Tue, 28 May 2024 17:59:19 GMT] Server:[nginx] Strict-Transport-Security:[max-age=31536000; includeSubDomains;] X-Content-Type-Options:[nosniff] X-Frame-Options:[DENY] X-Xss-Protection:[1; mode=block]] 0xc00020ee40 694 [] false false map[] 0xc0003a0400 0xc0003a6840}[{ [] false [{{0 0 0 0 0 0 0 0} 0 { } { } { } { }}] }]