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

Migrate GHA deployment workflows to ArgoCD triggering #1235

Merged
merged 12 commits into from
Jul 19, 2024
Merged

Conversation

i5okie
Copy link
Contributor

@i5okie i5okie commented Jul 3, 2024

- Update Build & Deploy Development workflow

  • Add logic to check for changes to the Helm chart files
  • Update trust-over-ip-configurations repo with chart/app version and image tag as necessary
  • Trigger remote workflow to propagate changes to the GitOps repo
  • Trigger ArgoCD Sync if no Helm chart changes, otherwise deploy with the chart changes

- Update Helm Chart Release workflow

  • Update Chart versions for Test and Production environments in the GitOps repo
  • Explicitly trigger ArgoCD Sync via webhook

- Miscellaneous

  • Update workflows and actions to use the latest stable publicly available actions

- Note

When updating the Helm chart default values, changes must be merged into trust-over-ip-configurations repo before merging a PR in this repo.


Workflow diagram-2024-07-17-184301


Addresses #1198
Part of bcgov/DITP-DevOps#158 (comment)

i5okie added 3 commits July 3, 2024 11:11
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie i5okie self-assigned this Jul 3, 2024
@i5okie i5okie requested a review from esune July 3, 2024 18:16
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie i5okie marked this pull request as ready for review July 8, 2024 20:29
esune
esune previously approved these changes Jul 9, 2024
Copy link
Member

@esune esune left a comment

Choose a reason for hiding this comment

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

Looks good, if @WadeBarnes can take a peek and confirm it would be optimal.
I logged an issue (see here) to further deal with updates and management of values files.

@i5okie i5okie requested a review from WadeBarnes July 10, 2024 17:48
@i5okie i5okie marked this pull request as draft July 12, 2024 16:32
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
i5okie added 3 commits July 17, 2024 11:03
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie i5okie marked this pull request as ready for review July 17, 2024 19:28
@i5okie i5okie marked this pull request as draft July 17, 2024 19:30
@i5okie i5okie marked this pull request as ready for review July 17, 2024 22:25
i5okie added 2 commits July 17, 2024 15:32
cleanup, update toipc repo, trigger argo

Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie i5okie requested a review from esune July 18, 2024 20:46
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@esune
Copy link
Member

esune commented Jul 19, 2024

@i5okie this is good to go, merge ahead when ready to move forward

@i5okie i5okie merged commit 5776538 into main Jul 19, 2024
4 checks passed
@esune esune deleted the feat/argo-workflows branch August 19, 2024 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants