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

Fixes #21023 - Update start-timeout to 90 #603

Merged
merged 1 commit into from
Oct 26, 2017
Merged

Fixes #21023 - Update start-timeout to 90 #603

merged 1 commit into from
Oct 26, 2017

Conversation

chris1984
Copy link
Member

No description provided.

@ekohl
Copy link
Member

ekohl commented Oct 25, 2017

My understanding the new plan was to lower the 600 second waiting of passenger to 90 so it matches the systemd default. @ehelms?

@ehelms
Copy link
Member

ehelms commented Oct 25, 2017 via email

@chris1984 chris1984 changed the title Fixes #21023 - Update httpd timeout to 90 Fixes #21023 - Update start-timeout to 90 Oct 25, 2017
@chris1984
Copy link
Member Author

chris1984 commented Oct 26, 2017

@ekohl can I get another review of this please? test failure looks unrelated

An error occurred while loading ./spec/acceptance/foreman_cli_spec.rb.
Failure/Error: require 'beaker-rspec/spec_helper'
Docker::Error::TimeoutError:
read timeout reached

@ekohl
Copy link
Member

ekohl commented Oct 26, 2017

@chris1984 see https://www.traviscistatus.com/incidents/v4jnp6nkgzqx for the docker timeout.

@chris1984
Copy link
Member Author

@ekohl tests are passing now, can I get a merge please?

@mmoll mmoll merged commit 951f30c into theforeman:master Oct 26, 2017
@mmoll
Copy link
Contributor

mmoll commented Oct 26, 2017

merged, thanks @chris1984!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants