Fixes #4770: Adding reset option to installer script. #37
Conversation
|
This will help QE to get a clean system without having to install the entire system, thanks. |
|
Scratch build if interested - http://koji.katello.org/koji/taskinfo?taskID=97139 |
|
Cool, will test it and give some feedback if I find something. Thank you |
| app_option( | ||
| '--reset', | ||
| :flag, | ||
| "Drop Katello database first? You will lose all data! Unfortunately we\n" + |
There was a problem hiding this comment.
"Drop Katello database first?"
Yes? No? I'm confused.
There was a problem hiding this comment.
Good point. Updated to reduce confusion.
|
I do notice that /var/lib/pulp/content is still populated with all the binary content along with /etc/puppet/environments/. pulp content is probably less of concern, but /etc/puppet/environments will cause entities to pop up in foreman, so I could see a stronger argument for removing that. |
|
Other than the above comment (which could be addressed separately), everything looks good to me, and ACK. |
|
@jlsherrill we could add an additional flag to indicate full data clearance if you feel that should be included as part of this PR or two separate flags - one for clearing puppet and one for clearing pulp content (or an issue to address each). |
|
@ehelms yeah, sounds good to me, i'll open a couple issues |
|
ACK |
Fixes #4770: Adding reset option to installer script.
Fixes #7802 - allow user to be passed into qpid module
Content views: support for view repositories
No description provided.