Last modified: 2013-09-11 22:46:24 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T53271, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51271 - Improving the way puppet roles are enabled
Improving the way puppet roles are enabled
Status: RESOLVED FIXED
Product: MediaWiki-Vagrant
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized enhancement
: ---
Assigned To: Ori Livneh
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-13 00:05 UTC by Zaran
Modified: 2013-09-11 22:46 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Zaran 2013-07-13 00:05:37 UTC
At the moment, there are two ways to enable a role in the puppet configuration: either by editing the Roles.yaml file in the root directory, or by editing puppet/manifests/sites.pp

I want to point out that this can be confusing for someone who is not familiar with the puppet machinery. As I am myself unfamiliar with it, here are the questions which naturally came to my mind:
* how does puppet know how to use the Roles.yaml file?
* which file has precedence over which?

Maybe (and this is just a suggestion) it would be less confusing to have the file sites.pp being automatically updated to reflect the content of Roles.yaml and vice versa?

Is the Roles.yaml file really needed? Is it much simpler for the user to edit Roles.yaml instead of puppet/manifests/sites.pp? Especially since it is just a matter of commenting/uncommenting lines?

There are probably other reasons that I am missing here, but in any case, the documentation should be updated to presently recommend editing the Roles.yaml file.
Comment 1 Ori Livneh 2013-07-29 21:01:10 UTC
I'm about to submit a patch that gets rid of Roles.yaml / Roles and instead provides a set of optional vagrant sub-commands for managing custom manifests. ('Optional' because the user may opt to hand-edit custom manifests in manifests.d instead.)

I apologize for being fickle, but I think I finally got it right. Feedback appreciated.
Comment 2 Gerrit Notification Bot 2013-07-29 21:02:05 UTC
Change 76625 had a related patch set uploaded by Ori.livneh:
Add Vagrant sub-commands for role management

https://gerrit.wikimedia.org/r/76625
Comment 3 Gerrit Notification Bot 2013-07-29 22:16:27 UTC
Change 76625 merged by jenkins-bot:
Add Vagrant sub-commands for role management

https://gerrit.wikimedia.org/r/76625
Comment 4 Ori Livneh 2013-07-29 22:16:58 UTC
Please re-open if you think the way this is handled is still messy.
Comment 5 Andre Klapper 2013-09-11 22:46:24 UTC
[mass-moving from Tools>MediaWiki-Vagrant to separate product. See bug 54041. Filter bugmail on this comment.]

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links