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

fix(devbox): Downgrade bosh-cli so that make deploy-autoscaler works on macOS #2942

Merged
merged 4 commits into from
May 22, 2024

Conversation

geigerj0
Copy link
Contributor

This PR brings #2619 also to the devbox environment.

@geigerj0 geigerj0 marked this pull request as ready for review May 22, 2024 09:25
@geigerj0 geigerj0 changed the title fix(devbox): Downgrade bosh-cli so that make deploy-autoscaler works on macOS fix(devbox): Downgrade bosh-cli so that make deploy-autoscaler works on macOS May 22, 2024
@geigerj0 geigerj0 changed the title fix(devbox): Downgrade bosh-cli so that make deploy-autoscaler works on macOS fix(devbox): Downgrade bosh-cli so that make deploy-autoscaler works on macOS May 22, 2024
@geigerj0 geigerj0 added exclude-from-changelog allow-acceptance-tests This label needs to be added to enable the acceptance tests to run. labels May 22, 2024
Copy link
Member

@silvestre silvestre left a comment

Choose a reason for hiding this comment

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

Please also

  1. Remove https://github.com/cloudfoundry/app-autoscaler-release/blob/main/renovate.json#L82-L93 as Renovate will keep the version up-to-date in .tool-versions
  2. Downgrade the version in .tool-versions as well, because https://github.com/cloudfoundry/app-autoscaler-release/blob/main/.github/workflows/asdf2devbox.yaml will try to keep the versions in sync

@geigerj0 geigerj0 added the bug label May 22, 2024
@geigerj0 geigerj0 requested a review from silvestre May 22, 2024 10:02
@geigerj0 geigerj0 force-pushed the downgrade-bosh-cli-for-devbox-env branch from 4a93b5e to 60d84c0 Compare May 22, 2024 11:13
Copy link

sonarcloud bot commented May 22, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

devbox.json Show resolved Hide resolved
@geigerj0 geigerj0 enabled auto-merge (squash) May 22, 2024 11:17
@geigerj0 geigerj0 merged commit 1a8862b into main May 22, 2024
31 checks passed
@geigerj0 geigerj0 deleted the downgrade-bosh-cli-for-devbox-env branch May 22, 2024 13:37
bonzofenix pushed a commit that referenced this pull request May 29, 2024
bonzofenix pushed a commit that referenced this pull request May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
allow-acceptance-tests This label needs to be added to enable the acceptance tests to run. bug exclude-from-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants