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

Hotfix add extra debug logs #157

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jbiers
Copy link
Member

@jbiers jbiers commented Jan 29, 2025

Related Issue: rancher/rancher#48175

Checklist

Please fill out this table to identify which fields need to be modified in your PR.

Under Status, either indicate Does Not Apply or Added to this PR.

Version to be incremented Why should this be modified? Status
version in rancher-project-monitoring package.yaml You modified the contents of the rancher-project-monitoring chart to make changes
helmProjectOperator.image.tag in prometheus-federator values.yaml Either you modified the rancher-project-monitoring chart or you modified the main.go file
appVersion in prometheus-federator Chart.yaml You modified the helmProjectOperator.image.tag in the above box
version in prometheus-federator Chart.yaml Either you modified the appVersion in the above box or you modified the contents of the prometheus-federator chart to make changes

@jbiers jbiers requested a review from a team as a code owner January 29, 2025 20:13
@jbiers jbiers force-pushed the hotfix-add-extra-debug-logging branch 2 times, most recently from f23b347 to 7e3e737 Compare February 1, 2025 05:25
@jbiers jbiers force-pushed the hotfix-add-extra-debug-logging branch from 7e3e737 to 0c79c79 Compare February 3, 2025 02:18
@jbiers jbiers force-pushed the hotfix-add-extra-debug-logging branch from 0c64743 to dae1057 Compare February 3, 2025 16:46
@jbiers jbiers force-pushed the hotfix-add-extra-debug-logging branch from 48d1bca to cba290d Compare February 3, 2025 16:58
@jbiers
Copy link
Member Author

jbiers commented Feb 3, 2025

@mallardduck @alexandreLamarre I've implemented the changes you suggested and fixed the failing CI jobs. I'm gonna send this branch to the customer having an issue which really needs more debug messages but if you think it's a good idea to merge this into main, we can do that as well.

I'm sending this as a "debug branch v1" since we'll likely need even more logs after this initial version, so any logs you might want to add are welcome.

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.

3 participants