Update from nginxinc to nginx #7103
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
documentation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7103 +/- ##
=======================================
Coverage 52.76% 52.77%
=======================================
Files 89 89
Lines 20831 20831
=======================================
+ Hits 10992 10994 +2
+ Misses 9382 9380 -2
Partials 457 457 ☔ View full report in Codecov by Sentry. |
pdabelf5
reviewed
Jan 10, 2025
| @@ -8,7 +8,7 @@ F5 NGINX Ingress Controller adheres to the support policy detailed in the follow | |||
|
|
|||
| After opening a support ticket, F5 staff will request additional information to better understand the problem. | |||
|
|
|||
| The [nginx-supportpkg-for-k8s](https://github.com/nginxinc/nginx-supportpkg-for-k8s) plugin collects the information needed by F5 Technical Support to assist with troubleshooting your issue. | |||
| The [nginx-supportpkg-for-k8s](https://github.com/nginx/nginx-supportpkg-for-k8s) plugin collects the information needed by F5 Technical Support to assist with troubleshooting your issue. | |||
There was a problem hiding this comment.
Is this being migrated as part of this project?
There was a problem hiding this comment.
not that I know of, reverting
pdabelf5
reviewed
Jan 10, 2025
pdabelf5
reviewed
Jan 10, 2025
| - Operator -- https://github.com/nginxinc/nginx-ingress-helm-operator | ||
| - Configuration examples -- https://github.com/nginx/kubernetes-ingress/tree/{{version}}/examples | ||
| - Helm Chart -- https://github.com/nginx/kubernetes-ingress/tree/{{version}}/deployments/helm-chart | ||
| - Operator -- https://github.com/nginx/nginx-ingress-helm-operator |
There was a problem hiding this comment.
Is the helm operator part of this migration project
There was a problem hiding this comment.
not initially, I'll revert the change
pdabelf5
reviewed
Jan 10, 2025
| @@ -26,15 +26,15 @@ The plugin may collect some or all of the following global and namespace-specifi | |||
|
|
|||
| This plugin **does not** collect secrets or coredumps. | |||
|
|
|||
| Visit the [project’s GitHub repository](https://github.com/nginxinc/nginx-supportpkg-for-k8s) for further details. | |||
| Visit the [project’s GitHub repository](https://github.com/nginx/nginx-supportpkg-for-k8s) for further details. | |||
There was a problem hiding this comment.
another reference to nginx-supportpkg-for-k8s
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Updates references from
nginxincChecklist
Before creating a PR, run through this checklist and mark each as complete.