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

Log output is invalid for regular builds #34

Open
bording opened this issue Nov 30, 2017 · 8 comments
Open

Log output is invalid for regular builds #34

bording opened this issue Nov 30, 2017 · 8 comments
Assignees

Comments

@bording
Copy link
Contributor

bording commented Nov 30, 2017

As originally reported in KirillOsenkov/MSBuildStructuredLog#136, it appears that the log levels are not being set correctly for regular builds. When I view a design-time log, everything is fine, but regular logs seem to be missing data. This was causing the log viewer to throw exceptions.

As mentioned in KirillOsenkov/MSBuildStructuredLog#136 (comment), manually increasing the log level of either UI option in VS appears to fix the issue.

Should this be something the extension is handling automatically? Which of the UI options should influence the log output?

@bording
Copy link
Contributor Author

bording commented Nov 30, 2017

Investigating a bit more, it seems to only occur for builds from SDK-style projects. Logs from legacy projects don't cause a problem for version 1.1.168 of the log viewer (the version before the workaround @KirillOsenkov added).

FYI I'm testing this with VS 2017 15.4.5 and the 2.0.3 SDK. I also have the latest 1.0.0.6230301 version of the extension.

@KirillOsenkov
Copy link
Member

Note that I've also seen wrong logs, I don't remember which project that was. But yes, this is definitely an issue.

@panopticoncentral panopticoncentral self-assigned this Dec 4, 2017
@bording
Copy link
Contributor Author

bording commented Dec 4, 2017

FYI, just tried this with VS 2017 15.5.0, 2.1.2 SDK, and extension 1.0.0.6232701, and no change in behavior from any of the updates.

@KirillOsenkov
Copy link
Member

Yup, I'm still seeing it too.

@clairernovotny
Copy link
Member

Is there a workaround, does this work from the command line instead? (for regular builds)?

@Pilchie
Copy link
Member

Pilchie commented Dec 11, 2017

it seems to only occur for builds from SDK-style projects.

I'm pretty sure I got an internal example where this wasn't the case today.

@panopticoncentral panopticoncentral removed their assignment Jun 11, 2019
@panopticoncentral
Copy link
Contributor

I'm closing this issue here because this isn't an issue with the project tools themselves. The problem is that CPS doesn't register loggers in the correct way in VS for regular builds because there's a missing API that lets them do that. I'm moving this to a VS issue and will work on resolving it there.

@bording
Copy link
Contributor Author

bording commented Jun 17, 2019

@panopticoncentral Glad to hear the problem has been identified and will be fixed, but will it be possible for you to report back here when the issue is resolved? I have no visibility into your internal issue trackers.

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

No branches or pull requests

5 participants