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

[DEVOPS-374] QOL updates to AKS deploy reusable workflow #105

Merged
merged 1 commit into from
Jan 18, 2024
Merged

[DEVOPS-374] QOL updates to AKS deploy reusable workflow #105

merged 1 commit into from
Jan 18, 2024

Conversation

ebronson68
Copy link
Contributor

@ebronson68 ebronson68 commented Jan 17, 2024 •
edited
Loading

DEVOPS-374
Summary Remove unused inputs from AKS deploy reusable workflow
Type Story Story
Status To Do
Points N/A
Labels -
---

Description

  • QOL updates to AKS deploy reusable workflow
    • Simplified inputs so that ones that don't ever change are just used in the workflow, rather than being set as an input.
    • Split Docker build/tag/push steps out for better visibility/granularity between the steps.
      image
    • Split out K8s secrets creation for environment variables and web authentication secrets
      image

Related Links

@github-actions github-actions bot added devops DevOps Team story New feature or request labels Jan 17, 2024
@ebronson68 ebronson68 marked this pull request as ready for review January 17, 2024 19:43
@ebronson68 ebronson68 requested a review from a team as a code owner January 17, 2024 19:43
@ebronson68 ebronson68 merged commit 59bd6d7 into main Jan 18, 2024
6 checks passed
@ebronson68 ebronson68 deleted the story/DEVOPS-374/remove-unused-inputs branch January 18, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
devops DevOps Team story New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ebronson68 @amutechtest