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

fix: update dashboard, etc, ingresss versions, fix broken features af… #752

Conversation

ikatlinsky
Copy link
Contributor

@ikatlinsky ikatlinsky commented Apr 29, 2024

This pull request addresses issues after merge of v2 chart from dev to master branch:

  1. Versions of etc, ingress controller and dashboard are outdated
  2. Some features that worked in v1 are now broken in v2 (apisix annotatation templates, custom secret keys for admin credentials)

Potentially there can be more issues for ones who will migrate from v1 to v2 version of the chart.

As a result of changes version of the apisix chart is updated to 2.7.0.

@ikatlinsky
Copy link
Contributor Author

@AlinsRan @Revolyssup @moonming please review and accept

@ikatlinsky
Copy link
Contributor Author

@AlinsRan @Revolyssup @moonming merged with 2.7.0 version of the chart and updated the version in this PR to 2.8.0

@ikatlinsky
Copy link
Contributor Author

@AlinsRan @Revolyssup @moonming up! thanks and sorry for pinging

@ikatlinsky
Copy link
Contributor Author

@AlinsRan @Revolyssup @moonming up

@ikatlinsky
Copy link
Contributor Author

@Revolyssup @moonming up

@ikatlinsky
Copy link
Contributor Author

@Gallardot @AlinsRan hi, any chances that this pr can be merged? thanks

@Gallardot Gallardot merged commit 97c26c0 into apache:master Jun 4, 2024
8 checks 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.

4 participants