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

vdk-cicd: cleanup cicd rules #1554

Merged
merged 5 commits into from
Jan 26, 2023

Conversation

murphp15
Copy link
Collaborator

@murphp15 murphp15 commented Jan 26, 2023

Why

We want to support scheduled jobs in the near future.
As a first step we should clean up what is there already.

What

This involves.

  1. rewriting only conditions as rules. which are more extensible in the future.
  2. upgrading helm.

How has this been tested.

I will babysit the pipelines after this is merged to identify any anomalies.

Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
@murphp15 murphp15 enabled auto-merge (squash) January 26, 2023 15:08
@murphp15 murphp15 merged commit 667eb82 into main Jan 26, 2023
@murphp15 murphp15 deleted the person/murphp15/use_rules_instead_of_only_clause branch January 26, 2023 17:10
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.

None yet

4 participants