Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Temporarily disable symweb indexing#10807

Merged
dagood merged 1 commit intodotnet:masterfrom
dagood:disable-indexing/master
Apr 7, 2017
Merged

Temporarily disable symweb indexing#10807
dagood merged 1 commit intodotnet:masterfrom
dagood:disable-indexing/master

Conversation

@dagood
Copy link
Member

@dagood dagood commented Apr 7, 2017

This step fails builds when the share is full, and the coreclr/master build takes up a lot of space. This share is also used by servicing branches and corefx. Disabling this step for now will take this pressure off the infra until we find a different mitigation.

Indexing on symweb isn't absolutely necessary. It is one symbol acquisition dev flow out of a few. For example, MyGet will still work.

/cc @MattGal

},
{
"enabled": true,
"enabled": false,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we please have a tracking issue to re-enable this when everything is back to normal?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea, created https://github.com/dotnet/coreclr/issues/10809 to track.

Copy link
Member

@gkhanna79 gkhanna79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM modulo the comment.

@dagood
Copy link
Member Author

dagood commented Apr 7, 2017

Merging, checked-in defs aren't used by CI.

@dagood dagood merged commit 628696b into dotnet:master Apr 7, 2017
@dagood dagood deleted the disable-indexing/master branch April 7, 2017 21:38
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants