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.

Sign up for GitHub

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

Jump to bottom

Included loop script and optimization #20

Merged
merged 7 commits into from
Mar 4, 2024
Merged

Included loop script and optimization #20

merged 7 commits into from
Mar 4, 2024

Conversation

mukrishn
Copy link
Collaborator

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

  • Copied loop script from old source
  • Fixed cluster operator role deletion, it was not using the right prefix
  • Let user to run workload script independently across all clusters using same uuid

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

@mukrishn mukrishn requested a review from dry923 November 22, 2023 20:10
@mukrishn mukrishn changed the title Loop script Included loop script and optimization Nov 22, 2023
@github-actions GitHub Actions
Copy link

This pull request has become stale and will be closed automatically within 7 days.

@github-actions github-actions bot added the stale label Feb 27, 2024
@github-actions github-actions bot removed the stale label Feb 29, 2024
dry923
dry923 approved these changes Feb 29, 2024
View reviewed changes
Copy link
Member

@dry923 dry923 left a comment

Choose a reason for hiding this comment

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

LGTM!

morenod
morenod approved these changes Mar 4, 2024
View reviewed changes
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

@morenod morenod merged commit ac90c07 into cloud-bulldozer:main Mar 4, 2024
1 of 3 checks passed
@mukrishn mukrishn mentioned this pull request Jul 9, 2024
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dry923 dry923 dry923 approved these changes

@morenod morenod morenod approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mukrishn @morenod @dry923