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

allow using args for start command in cmd_upgrade #4332

Merged
merged 2 commits into from
Mar 7, 2017
Merged

Conversation

riuvshin
Copy link
Contributor

@riuvshin riuvshin commented Mar 6, 2017

for now there is no way to pass any arg to start command on upgrade, but on our prod we must skip preflight checks because they will not work due to closed ports on firewall. those changes allow using upgrade --skip-backup --skip:preflight

What does this PR do?

allow using args for start command in cmd_upgrade

Changelog

allow using args for start command in cmd_upgrade

for now there is no way to pass any arg to start command on upgrade, but on our prod we must skip preflight checks because they will not work due to closed ports on firewall. those changes allow using `upgrade --skip-backup --skip:preflight`
@riuvshin riuvshin added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 6, 2017
@riuvshin riuvshin requested review from TylerJewell and benoitf March 6, 2017 16:36
@riuvshin riuvshin self-assigned this Mar 6, 2017
@riuvshin riuvshin requested a review from garagatyi March 6, 2017 16:39
@codenvy-ci
Copy link

Build finished.
Build success. $BUILD_URL

@codenvy-ci
Copy link

@riuvshin riuvshin merged commit 48ca887 into master Mar 7, 2017
@riuvshin riuvshin deleted the riuvshin-patch-1 branch March 7, 2017 09:30
@benoitf benoitf added this to the 5.5.0 milestone Mar 7, 2017
@JamesDrummond JamesDrummond mentioned this pull request Mar 17, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
* allow using args for start command in cmd_upgrade

for now there is no way to pass any arg to start command on upgrade, but on our prod we must skip preflight checks because they will not work due to closed ports on firewall. those changes allow using `upgrade --skip-backup --skip:preflight`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants