-
Notifications
You must be signed in to change notification settings - Fork 5k
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: Add controllerNamespace to .status of an application (#12964) #13081
feat: Add controllerNamespace to .status of an application (#12964) #13081
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #13081 +/- ##
=======================================
Coverage 48.98% 48.98%
=======================================
Files 249 249
Lines 43066 43067 +1
=======================================
+ Hits 21094 21098 +4
+ Misses 19864 19862 -2
+ Partials 2108 2107 -1
☔ View full report in Codecov by Sentry. |
Signed-off-by: Kevin Huber <kevin.huber@hotmail.com>
3535fba
to
1eaf3de
Compare
@jannfis can you PTAL? |
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @ItsKev!
…13081) Signed-off-by: Kevin Huber <kevin.huber@hotmail.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…13081) Signed-off-by: Kevin Huber <kevin.huber@hotmail.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Closes #12964
This pull request introduces the controller namespace to the application's .status. With this feature, users are able to trace back the controller that is responsible for managing the application.
Here are a few examples of how this feature can be useful: (as mentioned in #12964)
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Please see Contribution FAQs if you have questions about your pull-request.