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

feat: Parametrize runner instance launch configuration metadata options #348

Merged
merged 4 commits into from
Jul 22, 2021
Merged

feat: Parametrize runner instance launch configuration metadata options #348

merged 4 commits into from
Jul 22, 2021

Conversation

momirnikolic
Copy link
Contributor

Description

Parametrize usage of Gitlab runner agent instance AWS metadata service in launch configuration. This enables control of instance AWS IMDSv1 and IMDSv2 options and enables mitigation of AWS Security Hub issue related to IMDSv1 being enabled (EC2 instances should use Instance Metadata Service Version 2 (IMDSv2))

Migrations required

NO

Verification

Launch configuration AWS metadata service options change based on input variables given to the module.

Documentation

We use pre-commit to update the Terraform inputs and outputs in the documentation via terraform-docs. Ensure you have installed those components.

@momirnikolic momirnikolic changed the title Parametrize runner instance launch configuration metadata options feat: Parametrize runner instance launch configuration metadata options Jul 21, 2021
@npalm npalm self-requested a review July 21, 2021 17:42
Copy link
Collaborator

@npalm npalm left a comment

Choose a reason for hiding this comment

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

LGTM, will run a check asap

Copy link
Collaborator

@npalm npalm left a comment

Choose a reason for hiding this comment

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

Two minor changes suggested

variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
momirnikolic and others added 2 commits July 22, 2021 10:49
Co-authored-by: Niek Palm <[email protected]>
Co-authored-by: Niek Palm <[email protected]>
@momirnikolic
Copy link
Contributor Author

@npalm Thank you on a prompt review.
Suggested changes has been committed.

@npalm npalm merged commit a4406dc into cattle-ops:develop Jul 22, 2021
npalm added a commit that referenced this pull request Jul 22, 2021
…ns (#348)

* Parametrize runner instance launch configuration metadata options

* Parametrize runner instance launch configuration metadata options

* Update variables.tf

Co-authored-by: Niek Palm <[email protected]>

* Update variables.tf

Co-authored-by: Niek Palm <[email protected]>

Co-authored-by: Momir Nikolic <[email protected]>
Co-authored-by: Niek Palm <[email protected]>
semantic-releaser bot pushed a commit that referenced this pull request Jul 22, 2021
## [4.27.0](4.26.0...4.27.0) (2021-07-22)

### Features

* Parametrize runner instance launch configuration metadata options ([#348](#348)) ([92204ee](92204ee))
radimsuckr pushed a commit to radimsuckr/terraform-aws-gitlab-runner that referenced this pull request Jul 29, 2021
## [4.27.0](cattle-ops/terraform-aws-gitlab-runner@4.26.0...4.27.0) (2021-07-22)

### Features

* Parametrize runner instance launch configuration metadata options ([cattle-ops#348](cattle-ops#348)) ([92204ee](cattle-ops@92204ee))
semantic-releaser bot pushed a commit that referenced this pull request Aug 28, 2021
## [4.29.0](4.28.0...4.29.0) (2021-08-28)

### Features

* Allow configuring docker machine egress rules, see PR [#351](#351) for upgrade instructions ([845e018](845e018))
* Parametrize runner instance launch configuration metadata options ([#348](#348)) ([a4406dc](a4406dc))
* replace launch configuration with launch template ([#337](#337)) ([b805fb6](b805fb6))
* support for settings Sentry DSN ([#352](#352)) ([2a07466](2a07466))

### Bug Fixes

* Use better ressources names ([#356](#356)) ([817e040](817e040))
@semantic-releaser
Copy link
Contributor

🎉 This PR is included in version 4.29.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

npalm pushed a commit that referenced this pull request Oct 4, 2021
## [4.29.0](4.28.0...4.29.0) (2021-08-28)

### Features

* Allow configuring docker machine egress rules, see PR [#351](#351) for upgrade instructions ([845e018](845e018))
* Parametrize runner instance launch configuration metadata options ([#348](#348)) ([a4406dc](a4406dc))
* replace launch configuration with launch template ([#337](#337)) ([b805fb6](b805fb6))
* support for settings Sentry DSN ([#352](#352)) ([2a07466](2a07466))

### Bug Fixes

* Use better ressources names ([#356](#356)) ([817e040](817e040))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants