-
Notifications
You must be signed in to change notification settings - Fork 2.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
[mezmoexporter] Log HTTP errors #10875
Conversation
439edee
to
2dfcd70
Compare
79ab897
to
09126c0
Compare
I'm not sure why CI is asking me to run
|
87c09f3
to
03eb1e3
Compare
Can someone provide me with some guidance on this check? https://github.com/open-telemetry/opentelemetry-collector-contrib/runs/6862676992?check_suite_focus=true#step:9:1169 As noted above, I have run the suggested script and it results in no changes to commit. |
I see:
(sorry, didn't see) |
|
03eb1e3
to
1307bf3
Compare
a30803b
to
dcdd048
Compare
I could really use some guidance here. My Go experience is limited, and this project is quite complicated. As far as I can tell, there is nothing to be done. When I run |
I believe you just have to revert your changes on the top-level |
Thank you. That seems to have done it. I suspect what is happening is my IDE, GoLand, is updating that root file. What would I search for to find documentation on how this project is setup? If this is some sort of standard setup then maybe I can find a solution to make the IDE stop doing that. |
This change logs HTTP errors for increased visibility. We intend to follow up at a later date with more robust error handling. This change is meant to give users some insight into misconfigurations.
acc53e7
to
48ec275
Compare
I am not familiar with Goland but it may be related to our use of nested Go modules? There are some notes here about this: golang/go/issues/32394 |
Thank you. That gives me some direction on how to determine a resolution. |
This change logs HTTP errors for increased visibility. We intend to follow
up at a later date with more robust error handling. This change is
meant to give users some insight into misconfigurations.