cf-java-client icon indicating copy to clipboard operation
cf-java-client copied to clipboard
verified publisher icon cloudfoundry

Improve some integration-tests and cleanup /v2/info endpoint. fix #1229 #1249

Open Lokowandtg opened this issue 1 year ago • 5 comments

Replace calls to /v2/info endpoint with calls to root endpoint of API version 3. Skip integration tests that depend on routing if server does not provide it.

Lokowandtg avatar Nov 25 '24 18:11 Lokowandtg

Fixes cloudfoundry#1249 and cloudfoundry#1229

Lokowandtg avatar Nov 25 '24 18:11 Lokowandtg

Hey we've disabled V2 in our env in preparation for the V2 API deprecation which is scheduled for this month I think.

Think we need these changes merging in so we can exclusively use the v3 apis

thelangley avatar Jan 14 '25 09:01 thelangley

I totally agree. Who can do the review and merge? When this (and hopefully my other changes) are in, I will continue. For sure a lot of changes are required in the test suite, and I assume when v2 is missing, some more problems will show up.

Lokowandtg avatar Jan 14 '25 12:01 Lokowandtg

@anthonydahanne what do you think of the proposed changes?

thelangley avatar Jan 14 '25 13:01 thelangley

Looks like v2 api is now disabled by default on any opensource CF builds

https://github.com/cloudfoundry/cf-deployment/releases/tag/v47.0.0

@anthonydahanne, do you think this PR is a suitable approach?

thelangley avatar Feb 03 '25 07:02 thelangley