Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing Json Parsing Issue #2550

Merged
merged 5 commits into from
May 29, 2024
Merged

Fixing Json Parsing Issue #2550

merged 5 commits into from
May 29, 2024

Conversation

vprabhakar-px
Copy link
Contributor

@vprabhakar-px vprabhakar-px commented May 28, 2024

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 { } { } { } { }}] }]

Copy link
Contributor

@lsrinivas-pure lsrinivas-pure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@avinashsingh-px
Copy link
Member

As discussed on slack, merged the PR.

@avinashsingh-px avinashsingh-px merged commit 55dacfd into master May 29, 2024
3 of 5 checks passed
@vprabhakar-px vprabhakar-px deleted the PTX-24352-1 branch May 29, 2024 10:45
Madanagopal19 added a commit that referenced this pull request Jun 4, 2024
* kubevirt vendor updates for schedops (#2420)

Signed-off-by: Rohit-PX <[email protected]>
Co-authored-by: Rohit-PX <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>

* move total computation to before settting migration

Signed-off-by: pnookala-px <[email protected]>

* Retry GetVolumes() call

Signed-off-by: pnookala-px <[email protected]>

---------

Signed-off-by: pnookala-px <[email protected]>

* Create and attach 250 + FADA volumes in Longevity or scale testing automation (#2421)

Signed-off-by: Smit Jain <[email protected]>

* Fixing volume provisoner in case of back up test (#2437)

Co-authored-by: Leela Srinivas Siripurapu <[email protected]>

* 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 <[email protected]>

* fixing MultiVolumeMountsForSharedV4 for chos-level (#2446)

Co-authored-by: Leela Srinivas Siripurapu <[email protected]>

* [PWX-37026] Adding FBDAMultiTenancyBasicTest (#2419)

* [PWX-37026] Adding FBDAMultiTenancyBasicTest

Signed-off-by: Vinayak Shinde <[email protected]>

* [PWX-37026] Updated libopenstorage/openstorage dependancy

Signed-off-by: Vinayak Shinde <[email protected]>

* [PWX-37026] Added FBDAMultiTenancyBasicTest

Signed-off-by: Vinayak Shinde <[email protected]>

* [PWX-37026] Addressed review comments

Signed-off-by: Vinayak Shinde <[email protected]>

---------

Signed-off-by: Vinayak Shinde <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>

---------

Signed-off-by: Smit Jain <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>

* [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 <[email protected]>

* 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 <[email protected]>

---------

Signed-off-by: Vinayak Shinde <[email protected]>

* Run torpedo as a pod in anthos user cluster (#2491)

Signed-off-by: Smit Jain <[email protected]>

* PTX-24028 Vendor latest libopenstorage/operator package (#2489)

Signed-off-by: nikolaypopov <[email protected]>

* 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 <[email protected]>

* fbda with top test1

Signed-off-by: pnookala-px <[email protected]>

* fbda with top test1

Signed-off-by: pnookala-px <[email protected]>

* fbda with top test2

Signed-off-by: pnookala-px <[email protected]>

* sleep 30 seconds after deleting pvc

Signed-off-by: pnookala-px <[email protected]>

* create sc without top

Signed-off-by: pnookala-px <[email protected]>

* create sc without top

Signed-off-by: pnookala-px <[email protected]>

* fix spelling

Signed-off-by: pnookala-px <[email protected]>

* wait 3 mins only

Signed-off-by: pnookala-px <[email protected]>

* Refactored common code

---------

Signed-off-by: pnookala-px <[email protected]>
Co-authored-by: Vinayak Shinde <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: smarakala_pure <[email protected]>

* [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 <[email protected]>
Co-authored-by: smarakala_pure <[email protected]>

* minor fix for update node id function (#2515)

Co-authored-by: Leela Srinivas Siripurapu <[email protected]>

* [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 <[email protected]>
---------

Signed-off-by: Priyanshu Pandey <[email protected]>

* Adding cloudsnap delete event to longevity (#2444)

* Adding cloudsnap delete event to longevity

Signed-off-by: smarakala_pure <[email protected]>

* Adding params for px-security (#2527)

* Fixing OCP zones (#2516)

* Fixing OCP zones

* Fixing ASG for openshift

---------

Co-authored-by: Leela Srinivas Siripurapu <[email protected]>

* 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 <[email protected]>
Signed-off-by: pnookala-px <[email protected]>
Signed-off-by: Smit Jain <[email protected]>
Signed-off-by: Vinayak Shinde <[email protected]>
Signed-off-by: nikolaypopov <[email protected]>
Signed-off-by: smarakala_pure <[email protected]>
Signed-off-by: Priyanshu Pandey <[email protected]>
Co-authored-by: Rohit-PX <[email protected]>
Co-authored-by: Rohit-PX <[email protected]>
Co-authored-by: snigdha-px <[email protected]>
Co-authored-by: suhas <[email protected]>
Co-authored-by: Kshithij Iyer <[email protected]>
Co-authored-by: vprabhakar-px <[email protected]>
Co-authored-by: Leela Srinivas <[email protected]>
Co-authored-by: Leela Srinivas Siripurapu <[email protected]>
Co-authored-by: Adil Rahman <[email protected]>
Co-authored-by: sabrarhussaini <[email protected]>
Co-authored-by: pnookala-px <[email protected]>
Co-authored-by: jainsmit <[email protected]>
Co-authored-by: vumesh-px <[email protected]>
Co-authored-by: kphalgun-px <[email protected]>
Co-authored-by: Vinayak Shinde <[email protected]>
Co-authored-by: Leela Srinivas Siripurapu <[email protected]>
Co-authored-by: Geet-PX <[email protected]>
Co-authored-by: Mithun Koppal <[email protected]>
Co-authored-by: sgajawada-px <[email protected]>
Co-authored-by: Nikolay Popov <[email protected]>
Co-authored-by: px-kannavarapu <[email protected]>
Co-authored-by: Dhruv Bhatnagar <[email protected]>
Co-authored-by: Swarupa Vijaykumar <[email protected]>
Co-authored-by: Santhosh Marakala <[email protected]>
Co-authored-by: smarakala_pure <[email protected]>
Co-authored-by: Priyanshu Pandey <[email protected]>
Co-authored-by: sayalasomayajula-px <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants