Skip to content

Conversation

@jaer-tsun
Copy link
Contributor

What this PR does / why we need it:
Update operations should've been locked to prevent a race between other operations.

e.g.

namespace updates

npm deletes namespace
network policy updates
npm deletes network policy
npm adds network policy (fails because namespace was not created yet)
...
npm add namespace

@jaer-tsun jaer-tsun requested a review from csfmomo April 3, 2020 16:26
@matmerr
Copy link
Member

matmerr commented Apr 3, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jaer-tsun jaer-tsun merged commit 2427888 into Azure:master Apr 3, 2020
@jaer-tsun jaer-tsun deleted the lockNpmUpdates branch April 3, 2020 20:31
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.

4 participants