Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Fixes #13908 - use prefixed options in capsule and katello-devel #303

Merged
merged 1 commit into from Feb 26, 2016

Conversation

mbacovsky
Copy link
Contributor

No description provided.

@bbuckingham
Copy link
Member

The changes pass the 'eyeball' test. I'll run a test of installing katello, capsule and katello-devel to ensure all is well there too.

@bbuckingham
Copy link
Member

EDIT: Confirmed, the error below is unrelated to this PR.


EDIT: The error below does not appear to be related to this PR, but will test to confirm. I think it may be an issue with installing with katello-installer from git checkout.


The installer succeeds via katello-deploy for 'centos7-katello-devel'; however, it failed for 'centos7-katello'. The behavior that i see is below. Do you happen to see the same on your config?

==> centos7-katello-nightly: Complete!
==> centos7-katello-nightly: /vagrant/lib/katello_deploy/installer.rb:102:in `symlink': File exists - (/vagrant/katello-installer/config/katello.yaml, katello.yaml) (Errno::EEXIST)
==> centos7-katello-nightly:    from /vagrant/lib/katello_deploy/installer.rb:102:in `symlink'
==> centos7-katello-nightly:    from /vagrant/lib/katello_deploy/installer.rb:46:in `block in setup_config'
==> centos7-katello-nightly:    from /vagrant/lib/katello_deploy/installer.rb:45:in `chdir'
==> centos7-katello-nightly:    from /vagrant/lib/katello_deploy/installer.rb:45:in `setup_config'
==> centos7-katello-nightly:    from /vagrant/lib/katello_deploy/installer.rb:22:in `install'
==> centos7-katello-nightly:    from ./setup.rb:115:in `<main>'
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

@bbuckingham
Copy link
Member

ACK

@ehelms
Copy link
Member

ehelms commented Feb 26, 2016

Thanks @mbacovsky !

ehelms added a commit that referenced this pull request Feb 26, 2016
Fixes #13908 - use prefixed options in capsule and katello-devel
@ehelms ehelms merged commit ef06744 into Katello:master Feb 26, 2016
sthirugn added a commit to sthirugn/automation-tools that referenced this pull request Jun 7, 2016
sthirugn added a commit to sthirugn/automation-tools that referenced this pull request Jun 7, 2016
zjhuntin pushed a commit to zjhuntin/katello-installer that referenced this pull request Oct 30, 2018
Fixes #11110 - hammer content-view --help wrong remove-content descri…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants