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

fix typos in foreman-discovery-image-publish's SCM module #705

Merged
merged 1 commit into from Jun 22, 2018
Merged

fix typos in foreman-discovery-image-publish's SCM module #705

merged 1 commit into from Jun 22, 2018

Conversation

komidore64
Copy link
Contributor

also:

  • consolidating SCM and builder modules since nothing else uses them

also:
- consolidating SCM and builder modules since nothing else uses them
@lzap
Copy link
Member

lzap commented Jun 22, 2018

Can I get this merged? @mmoll the job blocks me from releasing FDI.

@mmoll mmoll merged commit 6dc27fa into theforeman:master Jun 22, 2018
@mmoll
Copy link
Contributor

mmoll commented Jun 22, 2018

merged, thanks @komidore64!

@lzap this is synced into Jenkins now, please test.

@lzap
Copy link
Member

lzap commented Jun 22, 2018

Nai, does not work. Another one:

+ sed -i 's/retryable(:on => Fog::Errors::TimeoutError, :tries => 200) do/retryable(:on => [Excon::Error::Timeout, Fog::Errors::TimeoutError], :tries => 200) do/' '/home/jenkins/.vagrant.d/gems/gems/vagrant-rackspace-0*/lib/vagrant-rackspace/action/create_server.rb'
sed: can't read /home/jenkins/.vagrant.d/gems/gems/vagrant-rackspace-0*/lib/vagrant-rackspace/action/create_server.rb: No such file or directory

@komidore64 I suggest instead doing regular rounds can you just fix this? The details (inputs) are in https://community.theforeman.org/t/fdi-build-jenkins-job-does-not-work/10113 so it's just one click. The job is supposed to spawn Fedora on Rackspace and executing build, the copy the result to downloads.theforeman.org.

@komidore64 komidore64 deleted the foreman-discovery-image-publish-fix branch June 22, 2018 11:42
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.

None yet

3 participants