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

bumped bosh-aws-cpi-release 99->101 to support new aws vm type disks #707

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

jackjoy-gds
Copy link
Contributor

What

bumped bosh-aws-cpi-release 99->101 to support new aws vm type disks

How to review

Either deploy to your own dev env or look at mine on dev04

Who can review

Any paas-dev


🚨⚠️ Please do not merge this pull request via the GitHub UI ⚠️🚨

@jackjoy-gds jackjoy-gds force-pushed the 186260603-support_new_aws_vm_type_disks branch 2 times, most recently from dbbd364 to c3343e5 Compare December 20, 2023 16:25
version: "99"
url: "https://bosh.io/d/github.com/cloudfoundry/bosh-aws-cpi-release?v=99"
sha1: "ffc4a06d6728d88eb108418f886f46428c2a1bf2"
version: "101"
Copy link
Contributor

Choose a reason for hiding this comment

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

You missed the comment above this section where it tells you to update the https://github.com/alphagov/paas-docker-cloudfoundry-tools/blob/main/bosh-cli-v2/Dockerfile file first.

Should we put in an # IMPORTANT!! there to make it more obvious?

@jackjoy-gds jackjoy-gds force-pushed the 186260603-support_new_aws_vm_type_disks branch from c3343e5 to 30c2481 Compare January 2, 2024 12:28
Copy link

@fearoffish fearoffish 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 e03656c into main Jan 17, 2024
1 check passed
@corlettb corlettb deleted the 186260603-support_new_aws_vm_type_disks branch January 17, 2024 09:44
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