Resolve bug in userdata, unsupported format. #5457

Merged
merged 3 commits into from May 26, 2016

Conversation

Projects
None yet
5 participants
Contributor

hoenirvili commented May 25, 2016

This is happening when the UserDataScript string is formatted
it adds in the process two new "\n\n". In order to prevent this we must
add the powershell header separate and cocat it. Bug:
(https://bugs.launchpad.net/juju-core/+bug/1585430).

(Review request: http://reviews.vapour.ws/r/4900/)

hoenirvili added some commits May 24, 2016

Resolve bug in userdata, unsupported format.
This is happening when the UserDataScript string is formatted
it adds in the process two new "\n\n". In order to prevent this we must
add the powershell header separate and cocat it. Bug:
(https://bugs.launchpad.net/juju-core/+bug/1585430).
Contributor

kat-co commented May 25, 2016

Please review the review at the link provided in the PR.

Member

axw commented May 26, 2016

$$fixes-1585430$$

Contributor

jujubot commented May 26, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 26, 2016

Build failed: Does not match ['fixes-1581074', 'fixes-1580418', 'fixes-1577945']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7872

Contributor

sinzui commented May 26, 2016

$$merge$$ fixes-1585430

Contributor

jujubot commented May 26, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 9ceef58 into juju:master May 26, 2016

@hoenirvili hoenirvili deleted the hoenirvili:deploy-fix branch May 26, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment