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

OCM-6285 | feat: Day1 SG support for HCP #810

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

hunterkepley
Copy link
Contributor

@hunterkepley hunterkepley commented Oct 8, 2024

What this PR does / why we need it:
Allows HCP clusters to have additional compute SGs on creation, following the introduced support for day 1 HCP SGs in CS/ROSA

Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story (OCM-xxxx):
Fixes #OCM-6285

Change type

  • New feature
  • Bug fix
  • Build
  • CI
  • Documentation
  • Performance
  • Refactor
  • Style
  • Unit tests
  • Subsystem tests

Checklist

@hunterkepley hunterkepley force-pushed the ocm-6285 branch 2 times, most recently from fbb91dd to 6052d7d Compare October 8, 2024 14:04
@hunterkepley
Copy link
Contributor Author

/retest

@hunterkepley
Copy link
Contributor Author

/test rosa-sts-advanced-critical-high-presubmit

@hunterkepley
Copy link
Contributor Author

/test rosa-sts-private-critical-high-presubmit

2 similar comments
@hunterkepley
Copy link
Contributor Author

/test rosa-sts-private-critical-high-presubmit

@hunterkepley
Copy link
Contributor Author

/test rosa-sts-private-critical-high-presubmit

@hunterkepley
Copy link
Contributor Author

/test rosa-sts-private-critical-high-presubmit

@hunterkepley
Copy link
Contributor Author

/retest-required

@hunterkepley hunterkepley force-pushed the ocm-6285 branch 2 times, most recently from 8a558e6 to ee77625 Compare October 14, 2024 14:48
@gdbranco
Copy link
Contributor

/lgtm
/approve

@hunterkepley
Copy link
Contributor Author

/retest-required

@hunterkepley
Copy link
Contributor Author

/retest-required

Copy link
Contributor

@robpblake robpblake left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Oct 16, 2024
Copy link

openshift-ci bot commented Oct 16, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gdbranco, robpblake

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 6b69a69 into terraform-redhat:main Oct 16, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants