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: create additional docker-machines firewall rule to account for multiple deployments overriding each other #5

Merged
merged 2 commits into from
Apr 5, 2022

Conversation

tomisinlalude-deimos
Copy link
Contributor

Screenshot 2022-04-05 at 15 25 52

@MeNsaaH MeNsaaH changed the title improvement: Modify docker machine firewall rule fix: create additional docker-machines firewall rule to account for multiple deployments overriding each other Apr 5, 2022
firewall.tf Show resolved Hide resolved
@MeNsaaH MeNsaaH merged commit 9c12581 into DeimosCloud:master Apr 5, 2022
MeNsaaH pushed a commit that referenced this pull request Apr 5, 2022
### [1.0.9](v1.0.8...v1.0.9) (2022-04-05)

### Bug Fixes

* create additional  docker-machines firewall rule to account for multiple deployments overriding each other ([#5](#5)) ([9c12581](9c12581))
@MeNsaaH
Copy link
Contributor

MeNsaaH commented Apr 5, 2022

This PR is included in version 1.0.9 🎉

github-actions bot pushed a commit to aayushprime/terraform-google-gitlab-ci-runner that referenced this pull request Oct 3, 2023
## 1.0.0 (2023-10-03)

### Features

* Added module for cache and kubernetes ([DeimosCloud#8](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/8)) ([0978d21](0978d21))
* Conditionally create node pool ([DeimosCloud#12](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/12)) ([2b8b64d](2b8b64d))

### Bug Fixes

* changed module.cache[0].cache_bucket_name -> module.cache.cache_bucket_name  ([DeimosCloud#10](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/10)) ([0d1fa07](0d1fa07))
* create additional  docker-machines firewall rule to account for multiple deployments overriding each other ([DeimosCloud#5](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/5)) ([9c12581](9c12581))
* fix cache name and optional docker-machine firewall creation ([DeimosCloud#13](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/13)) ([1d53300](1d53300))
* moved cache and gke-runner submodule into the folders directory ([DeimosCloud#11](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/11)) ([8443244](8443244))
* registration should use internal IP so that we can use tags for firewall rules ([DeimosCloud#4](https://github.com/aayushprime/terraform-google-gitlab-ci-runner/issues/4)) ([8926850](8926850))
* update runner machine image to ubuntu-2004 to fix docker installation ([093ba99](093ba99))
* use internal Ip for dummy runner so that firewall rule works ([f57a91f](f57a91f))
* use the same image for spinning up dummy machine and update default runner machine image to COS ([7c24b17](7c24b17))
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