-
Notifications
You must be signed in to change notification settings - Fork 18
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
packit: tag cloud resources in Testing Farm #130
Conversation
Also let's wait for tests, so we can verify it is doing what it should be |
@thrix Thanks for PR. Can you please modify little bit commit message? The commit message should begin with one of the following category keyword: "build,ci,chore,docs,feat,fix,perf,refactor,revert,style,test" |
Jenkins test this please. |
5a70d68
to
ae2f000
Compare
hopefully ok now |
.packit.yml
Outdated
settings: | ||
provisioning: | ||
tags: | ||
BusinessUnit: RedHatInsights@public |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you choose this business unit? According to this documentation https://packit.dev/docs/configuration/examples#tests it should be RHEL sst name. Our sst name is something different.
ae2f000
to
e56036b
Compare
This way we can track the cloud costs for the team and also provide more detailed team specific metrics. Signed-off-by: Miroslav Vadkerti <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
This way we can track the cloud costs for the team and also provide more detailed team specific metrics.