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

feat: nginx v1.11.2 and prepare release v2.3.3 #129

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Conversation

ralgozino
Copy link
Member

Changes included in this PR:

  • Bump nginx ingress controller version to v1.11.2
  • Add release notes for v2.3.3 of the module including the new nginx
    version
  • Update readmes for v2.3.3 module release
  • Comp Matrix: add new v2.3.3 version
  • Comp Matrix: nove old 1.x version to the legacy section
  • Maintenance guide: updated the guide to generate manifests that are closer to the ones in the module, still needs work.

Other changes:

  • Cleaned up unused files to avoid confusion for new comers.

The new version of nginx has been manually testes in a KFD 1.29.2 cluster with dual ingress. All working as expected.

I'll tag a release candidate on this PR to trigger the e2e tests.

Note: the new v1.11.2 image has already be synced to our registry.

- Bump nginx ingress controller version to v1.11.2
- Add release notes for v2.3.3 of the module including the new nginx
version
- Update readmes for v2.3.3 module release
- Comp Matrix; add new v2.3.3 version
- Comp Matrix: nove old 1.x version to the legacy section
delete unused files from the repository that could cause confusion.
sbruzzese902
sbruzzese902 previously approved these changes Aug 21, 2024
Copy link
Contributor

@sbruzzese902 sbruzzese902 left a comment

Choose a reason for hiding this comment

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

LGTM! 😄

@ralgozino
Copy link
Member Author

ralgozino commented Aug 23, 2024

I don't know why it is not showing here, but CI is passing for both the push and the release candidate tag events:

https://ci.sighup.io/sighupio/fury-kubernetes-ingress/1090

Probably something is going on with the webhooks, even though they seem fine from the GitHub's side.

I'll merge.

@ralgozino ralgozino merged commit 2e1ec6f into main Aug 23, 2024
@ralgozino ralgozino deleted the feat-nginx-1.11.2 branch August 23, 2024 10:19
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