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

test: auto correct grafana port in test vagrant vms #13202

Merged
merged 1 commit into from Sep 17, 2020

Conversation

nebril
Copy link
Member

@nebril nebril commented Sep 17, 2020

Not having auto_correct set to true caused concurrent vagrant envs
to conflict on host port 3000 which caused builds to fail

Not having auto_correct set to true caused concurrent vagrant envs
to conflict on host port 3000 which caused builds to fail

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril requested a review from a team as a code owner September 17, 2020 09:45
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 17, 2020
@nebril
Copy link
Member Author

nebril commented Sep 17, 2020

test-missed-k8s

@nebril
Copy link
Member Author

nebril commented Sep 17, 2020

test-me-please

@nebril nebril added the release-note/ci This PR makes changes to the CI. label Sep 17, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 17, 2020
@aanm
Copy link
Member

aanm commented Sep 17, 2020

I'm going to merge this since all required tests have passed. The #13199 will run the tests for the missed k8s versions

@aanm aanm merged commit cbf8e00 into master Sep 17, 2020
@aanm aanm deleted the pr/auto-correct-grafan-port branch September 17, 2020 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants