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

Add Additional tags to Hypershift Clusters #492

Merged
merged 2 commits into from
Oct 17, 2022

Conversation

krishvoor
Copy link
Member

Description

This PR adds the following tags to the HyperShift clusters

  1. User:${GITHUB_USER}
  2. {MGMT_CLUSTER_NAME}:{HOSTED_CLUSTER_NAME}

Signed-off-by: Krishna Harsha Voora krvoora@redhat.com

This PR adds following tags to the HyperShift clusters
1) User:${GITHUB_USER}
2) {MGMT_CLUSTER_NAME}:{HOSTED_CLUSTER_NAME}

Signed-off-by: Krishna Harsha Voora <krvoora@redhat.com>
@krishvoor
Copy link
Member Author

Hi @dry923 @morenod @mukrishn 👋
ptal

Copy link
Collaborator

@morenod morenod left a comment

Choose a reason for hiding this comment

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

LGTM

@krishvoor
Copy link
Member Author

Hi @mukrishn as discussed, updated the tags, ptal

@smalleni
Copy link
Collaborator

smalleni commented Oct 12, 2022

@krishvoor @mukrishn @morenod Adding User tag is useful for airflow but probably not so much for e2e. Here, folks usually run from the upstream repo itself and not the fork. So most likely, the User is always cloud-bulldozer. The user tag is more useful for airflow playgrounds IMO

@krishvoor
Copy link
Member Author

krishvoor commented Oct 13, 2022

@smalleni I hear you, the original scope of the project was to add tags to track hosted cluster-info.
I agree the User tag should go into the airflow playground.

When working with e2e-benchmarking-->hypershift script, we definitely need a way to know which user spun the hosted cluster.

Signed-off-by: Krishna Harsha Voora <krvoora@redhat.com>
@mukrishn mukrishn merged commit 4f4abfe into cloud-bulldozer:master Oct 17, 2022
@krishvoor krishvoor deleted the hypershift-tags branch October 18, 2022 01:07
vishnuchalla pushed a commit that referenced this pull request Sep 6, 2023
* Add Additional tags to Hypershift Clusters

This PR adds following tags to the HyperShift clusters
1) User:${GITHUB_USER}
2) {MGMT_CLUSTER_NAME}:{HOSTED_CLUSTER_NAME}

Signed-off-by: Krishna Harsha Voora <krvoora@redhat.com>

* Updates the tags

Signed-off-by: Krishna Harsha Voora <krvoora@redhat.com>

Signed-off-by: Krishna Harsha Voora <krvoora@redhat.com>
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.

4 participants