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

keeping track of provision.sh runs #1753

Merged
merged 1 commit into from
Nov 10, 2017

Conversation

jackfrancis
Copy link
Member

and only executing provision.sh if we haven’t already

What this PR does / why we need it: ensures that provision script only runs once

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1752

Release note:

only execute provision script once

and only executing provision.sh if we haven’t already
@ghost ghost assigned jackfrancis Nov 10, 2017
@ghost ghost added the in progress label Nov 10, 2017
Copy link
Contributor

@anhowe anhowe left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@JackQuincy JackQuincy left a comment

Choose a reason for hiding this comment

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

LGTM, not sure how much that matters as we wrote it together

@jackfrancis
Copy link
Member Author

@JackQuincy I appreciate your approval even in excess :)

@jackfrancis jackfrancis merged commit b5eb43b into Azure:master Nov 10, 2017
@ghost ghost removed the in progress label Nov 10, 2017
@jackfrancis jackfrancis deleted the conditional-provision-script branch November 10, 2017 22:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clusters rebooting all nodes at the same time
3 participants