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

feat: check nginx chart needs to be installed before installing #62

Merged

colesnodgrass merged 2 commits into main from cole/make-nginx-smarter

Jul 26, 2024

Member

colesnodgrass commented Jul 25, 2024

fix https://github.com/airbytehq/airbyte-internal-issues/issues/8843
- workaround the problem with nginx upgrade failing by avoiding the upgrade entirely
- if the installed nginx chart version and appversion match the chart, skip the upgrade


          check nginx chart needs to be installed before installing

5d1fd83

colesnodgrass requested review from perangel and a team

July 25, 2024 22:38


          fix tests

667a928

perangel approved these changes

View reviewed changes

Collaborator

perangel left a comment

LGTM

bgroff approved these changes

View reviewed changes

colesnodgrass merged commit 7eda42f into main

2 checks passed

colesnodgrass deleted the cole/make-nginx-smarter branch

July 26, 2024 20:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet