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

Update .packit.yaml config #1252

Merged
merged 4 commits into from
Jun 10, 2024
Merged

Conversation

mm0ran
Copy link
Member

@mm0ran mm0ran commented Jun 6, 2024

  • Remove unused RHSM_REPOS and LEAPPDATA_BRANCH
  • Remove enabling repos step from .packit.yaml, it was moved to TF, where it logically belongs to.
  • Add required distro_target context
  • Update ENV variables for RHUI tests
  • Update plan filters
  • Enable upgrade path to 8.10 on AWS

Bundles the code from these PRs, and adds additional changes:

mkluson and others added 3 commits June 6, 2024 08:37
- Remove unused `RHSM_REPOS` and `LEAPPDATA_BRANCH`
- Update ENV variables for RHUI tests
- enable upgrade path to 8.10 on AWS
Remove enabling repos step from .packit.yaml, it was moved to TF, where
it logically belongs to.
@mm0ran mm0ran force-pushed the update_packit_config branch from a438f31 to 9de4488 Compare June 7, 2024 07:16
.packit.yaml Outdated
@@ -135,19 +123,6 @@ jobs:
identifier: sanity-abstract-7to8-aws
# NOTE(ivasilev) Unfortunately to use yaml templates we need to rewrite the whole tf_extra_params dict
Copy link
Member

Choose a reason for hiding this comment

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

this comment seems connected to the deleted lines

.packit.yaml Show resolved Hide resolved
- Remove comment connected to deleted lines
- Add BusinessUnit to all environments
@mm0ran mm0ran requested a review from fernflower June 7, 2024 12:07
Copy link
Member

@fernflower fernflower left a comment

Choose a reason for hiding this comment

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

Lgtm!

@fernflower fernflower merged commit 91f4e3b into oamg:master Jun 10, 2024
15 checks passed
@pirat89 pirat89 added the changelog-checked The merger/reviewer checked the changelog draft document and updated it when relevant label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-checked The merger/reviewer checked the changelog draft document and updated it when relevant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants