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: Implements placement options and license specifications #607

Merged
merged 5 commits into from
Nov 27, 2023

Conversation

mfortin
Copy link
Contributor

@mfortin mfortin commented Oct 21, 2023

Description

Implements placement options and license specifications

Issues Resolved

#591
tenancy: dedicated may not be sufficient for all use cases. Adding options to set placement host resource group arn and license configurations may be required in some advanced use cases.

Type of Change

feat: Add placement options and License Specifications

Check List

@mfortin mfortin marked this pull request as ready for review October 21, 2023 18:36
@damacus damacus changed the title Implements issue 591 feat: Implements placement options and license specifications Nov 27, 2023
@damacus
Copy link
Contributor

damacus commented Nov 27, 2023

Hey @mfortin this looks pretty reasonable to me!

Can you run Chefstyle over the codebase and re-push, please?

@mfortin
Copy link
Contributor Author

mfortin commented Nov 27, 2023

@damacus Done

@damacus damacus enabled auto-merge (squash) November 27, 2023 14:18
@damacus damacus merged commit 9a269b4 into test-kitchen:main Nov 27, 2023
3 of 5 checks passed
damacus added a commit that referenced this pull request Nov 27, 2023
damacus added a commit that referenced this pull request Nov 27, 2023
@damacus
Copy link
Contributor

damacus commented Nov 27, 2023

@mfortin apologies I turned on auto-merge, before we had protected the main branch. Can you repush the PR please?

It looks like there is a problem in the code that the unit tests have picked up.

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