Skip to content

Conversation

@radsoulbeard
Copy link
Member

@radsoulbeard radsoulbeard commented Jan 19, 2021

  • update mta plugin to 2.5.1
  • remove create service plugin as contained in cf cli v7

fixes #31

- update mta plugin to 2.5.1
- remove create service plugin as contained in cf cli v7
@radsoulbeard radsoulbeard requested a review from fwilhe January 19, 2021 14:58
@radsoulbeard radsoulbeard self-assigned this Jan 20, 2021
@fwilhe
Copy link
Contributor

fwilhe commented Jan 20, 2021

Hi @radsoulbeard

are you sure this is not a breaking change for existing pipelines? I don't have the time to do an in depth review, and I'm not an active CF user (anymore), so I'm probably not the best reviewer here.

Looping in @marcusholl @CCFenner

Can you judge if this breaks anything?

@marcusholl
Copy link
Member

Can you judge if this breaks anything?

No. Not close enough to cf use-cases. We should have test cases, I suppose ...

@radsoulbeard
Copy link
Member Author

Hi @radsoulbeard

are you sure this is not a breaking change for existing pipelines? I don't have the time to do an in depth review, and I'm not an active CF user (anymore), so I'm probably not the best reviewer here.

Looping in @marcusholl @CCFenner

Can you judge if this breaks anything?

Creating a new image version does not mean to change the defaults in the Piper library immediately. The integration tests passed.

@marcusholl
Copy link
Member

@radsoulbeard

does not mean to change the defaults in the Piper library immediately

Can you clarify that? As far as I can see we have configured ppiper/cf-cli here and some lines below. Since there is no explicit version we use latest, I suppose. That means if there is a new cf-cli docker image that-one will be used. Am I wrong?

Maybe we recommend to the customers to configure fixed versions, but by default there is no fixed version.

@radsoulbeard
Copy link
Member Author

@marcusholl
You are right, that will pull latest. We need to test that carefully.

@radsoulbeard
Copy link
Member Author

testResults.txt
I tested all additional steps with the new version. cloudFoundryCreateSpace, cloudFoundryCreateService, cloudFoundryCreateServiceKey, cloudFoundryDeleteService and cloudFoundryDeleteSpace still works.

@radsoulbeard
Copy link
Member Author

As discussed, SAP/jenkins-library defaults are now on a dedicated tag v5
Temporary fix ppiper/cf-cli version SAP/jenkins-library#2541

Copy link
Member

@OliverNocon OliverNocon left a comment

Choose a reason for hiding this comment

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

LGTM

@radsoulbeard radsoulbeard changed the title update cf cli to v7 update cf cli to version 7.2 Jan 28, 2021
@radsoulbeard radsoulbeard merged commit 8f452fd into master Jan 28, 2021
@radsoulbeard radsoulbeard deleted the pr/updatecfcliv7 branch January 28, 2021 11:17
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.

cf-cli is still the deprecated v6

5 participants