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

Investigate missing symbol publishing for all repos and branches using V3 #8015

Closed
1 of 2 tasks
riarenas opened this issue Oct 8, 2021 · 19 comments
Closed
1 of 2 tasks
Assignees
Labels
Detected By - Customer Issue was reported by a customer Needs Triage A new issue that needs to be associated with an epic. Ops - First Responder

Comments

@riarenas
Copy link
Member

riarenas commented Oct 8, 2021

  • This issue is blocking

This is blocking msbuild insertion into VS.

  • This issue is causing unreasonable pain

There seems to have been a regression in symbol publishing that might be related to a661399

The logs for the latest maestro publishing build no longer shows any indication that symbols are actually being pushed to the symbol servers:

https://dev.azure.com/devdiv/DevDiv/_build/results?buildId=5309117&view=logs&j=ba23343f-f710-5af9-782d-5bd26b102304&t=6e277ba4-1c1e-552d-b96f-db0aeb4be20a.

@ghost ghost added the Needs Triage A new issue that needs to be associated with an epic. label Oct 8, 2021
@riarenas riarenas added the Detected By - Customer Issue was reported by a customer label Oct 8, 2021
@riarenas
Copy link
Member Author

riarenas commented Oct 8, 2021

@epananth FYI

@benvillalobos
Copy link
Member

@dotnet/kitten FYI

@hoyosjs
Copy link
Member

hoyosjs commented Oct 10, 2021

This is actually affecting all repos - runtime is also having this issue.

@hoyosjs
Copy link
Member

hoyosjs commented Oct 11, 2021

This is also affecting release branches (apparently publish using darc always points to the yml of the main branch).

@hoyosjs hoyosjs changed the title Investigate missing symbol publishing for msbuild Investigate missing symbol publishing for all repos and branches using V3 Oct 11, 2021
@hoyosjs
Copy link
Member

hoyosjs commented Oct 11, 2021

https://dev.azure.com/dnceng/internal/_build/results?buildId=1412669

A revert of the specified commit didn't help. Something else seems to be causing this issue.

@ViktorHofer
Copy link
Member

Official builds targeting both 5.0 and 6.0 are failing for dotnet/runtime presumably because of this. Ie https://dev.azure.com/dnceng/internal/_build/results?buildId=1412894&view=logs&j=ba23343f-f710-5af9-782d-5bd26b102304&t=6e277ba4-1c1e-552d-b96f-db0aeb4be20a

@hoyosjs
Copy link
Member

hoyosjs commented Oct 11, 2021

This one is different - and new...

@ViktorHofer
Copy link
Member

Could this be the reason for the missing symbols here: dotnet/runtime#57259 (comment)?

@michellemcdaniel
Copy link
Contributor

@alexperovich Is it possible something about the new publishing work you're doing affected symbols publishing?

@riarenas riarenas self-assigned this Oct 12, 2021
@michellemcdaniel
Copy link
Contributor

michellemcdaniel commented Oct 12, 2021

My best guess on this, from just scanning the logs, is that the serversToPublish list (ie the list of servers to publish to) is empty.

@riarenas
Copy link
Member Author

Yep, that's a good assessment. I'll see if I can find where the targetFeedConfigs disconnect is happening.

@alexperovich
Copy link
Member

With my publishing changes we made a specific decision to publish symbols to azure storage instead of the nuget feeds they used to be in. I didn't touch any of the code that uploads to msdl or symweb, but potentially it got disconnected somewhere. I will check.

@riarenas
Copy link
Member Author

This call seems to be returning empty:

Dictionary<string, string> serversToPublish =
GetTargetSymbolServers(feedConfigsForSymbols, msdlToken, symWebToken);
so there's probably some disconnect around the feedConfigsForSymbols

@alexperovich
Copy link
Member

Yep, it looks like thats the problem. "SymbolTargetType" is unset for the new feed configs. I will push a fix.

@alexperovich
Copy link
Member

#8024

@riarenas
Copy link
Member Author

Thanks Alex. I'll make sure to get the fix into the publishing pipeline as soon as it's available.

@riarenas
Copy link
Member Author

#8027 should fix this once merged. The validation build at https://dev.azure.com/dnceng/internal/_build/results?buildId=1416538&view=logs&j=ba23343f-f710-5af9-782d-5bd26b102304&t=6e277ba4-1c1e-552d-b96f-db0aeb4be20a published symbols again as it used to.

@riarenas
Copy link
Member Author

Change is merged and newer runs should publish symbols again. I'll look at some runs for the publishing pipeline tomorrow and close this if things look good.

@riarenas
Copy link
Member Author

Newer runs are attempting to publish symbols. There's a new problem I'm seeing during symbol publishing so I opened #8033 for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Detected By - Customer Issue was reported by a customer Needs Triage A new issue that needs to be associated with an epic. Ops - First Responder
Projects
None yet
Development

No branches or pull requests

6 participants