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

Revert and bump minio #478

Merged
merged 2 commits into from
Mar 7, 2024
Merged

Revert and bump minio #478

merged 2 commits into from
Mar 7, 2024

Conversation

accenture-mikeyc
Copy link
Contributor

Ticket Link #fill_in

Remove if not applicable

Description

What was changed

Reminders

  • Did you up the relevant chart version numbers? (If appropriate)
    • If you up a chart version within urban-os, have you also upped the urban-os chart version itself?
    • If charts within the urban-os chart (andi, etc) have been updated, have you run helm dependency update in /charts/urban-os and commited the Chart.lock file?
  • Does helm template . -f values.yaml pass? (Checks for default values provided in chart and catches other errors)
  • Do you have git hooks installed? (See README.md to install)
  • If global values were altered, are they included as chart default values?
    • Are they also specified in the urbanos chart values file?
  • If references to external charts were added:
    • Was the github release action updated to helm update {new_thing} it's dependencies?
    • Was the deploy repo -u flag updated to helm update {new_thing} to ensure it's not left out of deployments?

Copy link
Contributor

@AFarrAccenture AFarrAccenture left a comment

Choose a reason for hiding this comment

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

.

@accenture-mikeyc accenture-mikeyc merged commit d3f154b into master Mar 7, 2024
1 check passed
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.

2 participants