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

Updates to bosh-cli-v2 #291

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Updates to bosh-cli-v2 #291

merged 2 commits into from
Aug 17, 2023

Conversation

corlettb
Copy link
Contributor

@corlettb corlettb commented Aug 16, 2023

What

Version updates to bosh-cli-v2 container:

  • bump bosh cli version to 7.3.1
  • bump bosh aws cpi to version 99
  • bump credhub cli version to 2.9.19
  • bump yq version to 4.35.1

Clean up packages folder at the end of the bosh create-env.

If we don't clean the packages and we have a different cpi version built upstream we can get "idempotent" issues.

For example:

mkdir: cannot create directory '/root/.bosh/installations/44f01911-a47a-4a24-6ca3-a3109b33f058/packages/aws-cpi-ruby-3.1/bosh': File exists

Example build failure

How to review

Look at the changes.
These changes have been test against dev-04. Both paas-bootstrap and paas-cf.

@corlettb corlettb force-pushed the 185698054-upgrade-bosh-cpi branch 3 times, most recently from efd116a to 6a30759 Compare August 16, 2023 09:55
- bump bosh cli version to 7.3.1
- bump bosh aws cpi to version 99
- bump credhub cli version to 2.9.19
- bump yq version to 4.35.1
If we don't clean the packages and the paas-bootstrap project uses a different version we can get "idempotent" issues.

For example:

mkdir: cannot create directory '/root/.bosh/installations/44f01911-a47a-4a24-6ca3-a3109b33f058/packages/aws-cpi-ruby-3.1/bosh': File exists
@corlettb corlettb changed the title Bump bosh-aws-cpi-release to version 99 Updates to bosh-cli-v2 Aug 17, 2023
@corlettb corlettb marked this pull request as ready for review August 17, 2023 07:51
Copy link
Contributor

@jackjoy-gds jackjoy-gds left a comment

Choose a reason for hiding this comment

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

LGTM

@corlettb corlettb merged commit acff07c into main Aug 17, 2023
28 checks passed
@corlettb corlettb deleted the 185698054-upgrade-bosh-cpi branch August 17, 2023 10:12
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.

2 participants