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

(GH-223) .travis.yml does not deploy by default #224

Merged
merged 1 commit into from
Apr 26, 2019

Conversation

ghoneycutt
Copy link
Contributor

Without this patch, the TravisCI configuration will be broken and add a
deployment section with the incorrect password and without documentation
to explain how to do deploys correctly.

@ghoneycutt
Copy link
Contributor Author

Fixes #223

@rtib
Copy link
Contributor

rtib commented Mar 28, 2019

I'm okay with the idea of having deployment setup optional. But, I think the feature toggle is not really necessary.

How about this: drop the deploy config key and enable the deployment section if user and secure are set.
This:

  • does not require any change for modules having deploy already in place
  • reduces config_defaults.yml
  • simplifies .sync.yml for enabling deploy by requiring unavoidable settings only
  • keeps the .travis.yml.erb template still readable

BTW, the default setting user: puppet favours a single publisher. While I really appreciate Voxpupuli a lot, I think this setting is wrong at pdk-templates.

Look at ae4c4e6.

@ghoneycutt
Copy link
Contributor Author

Good idea, though I don't know how to implement it.

@ghoneycutt
Copy link
Contributor Author

@scotje could you please review

@rtib
Copy link
Contributor

rtib commented Mar 28, 2019

Good idea, though I don't know how to implement it.

Here is the example

@scotje
Copy link
Contributor

scotje commented Apr 1, 2019

I agree with the change suggested by @rtib. After that I'm +1 on merging this.

scotje
scotje previously approved these changes Apr 3, 2019
@scotje scotje requested a review from rodjek April 3, 2019 17:53
rodjek
rodjek previously approved these changes Apr 26, 2019
Without this patch, the TravisCI configuration will be broken and add a
deployment section with the incorrect password and without documentation
to explain how to do deploys correctly.

Co-authored-by: Garrett Honeycutt <code@garretthoneycutt.com>
Co-authored-by: Tibor Répási <rtib@users.noreply.github.com>
@rodjek rodjek dismissed stale reviews from scotje and themself via ed32c02 April 26, 2019 17:55
@rodjek rodjek merged commit 963fbb5 into puppetlabs:master Apr 26, 2019
@ghoneycutt ghoneycutt deleted the gh223_travis_deploy branch April 27, 2019 15:56
@scotje scotje added this to the PDK 1.11.0.0 milestone Jun 25, 2019
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