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

[DONT MERGE] Destroy VM and "retry on failure" policy changes #126

Closed
wants to merge 4 commits into from

Conversation

Kami
Copy link
Member

@Kami Kami commented Feb 10, 2016

This adds back changes from #119 and #118 which were reverted to get things working / passing.

Keep in mind that we still need to update all the workflows which utilize ec2_run_instances because the result format has changed.

This should only be merged after v1.3.1 release is out and after the AWS pack stuff and existing workflows have been updated.

@Kami Kami changed the title [DONT MERGE] Destroy vm changes [DONT MERGE] Destroy VM and "retry on failure" policy changes Feb 10, 2016
manasdk added 2 commits February 10, 2016 11:48
* master: (26 commits)
  Try decreasing sleep even further.
  Remote runner doesn't like pkill, sigh.
  Decrease timeout.
  Switch back to pkill, use ; instead of &&.
  Use different approach instead of pkill.
  Make sure all the action runner processes are actually killed.
  Add additional check which checks that all the services are running post upgrade restart.
  Sleep before restart in the post upgrade restart.
  Add some sleep before and after restart.
  hosts to hostname
  Add missing param
  Fix param and on-success task
  Add build parameter since it is required downstream
  revert destroy_vm change to upgrade test workflow
  Write proper YAML
  Fix names
  fix upgrade section
  Fix typos or workflow errors
  Remove blank line.
  Fix package names, improve version checks (python-st2client packages follows a different version format, sigh :/).
  ...
@Kami
Copy link
Member Author

Kami commented Feb 11, 2016

Merged into #128, will continue in that PR / branch.

@Kami Kami closed this Feb 11, 2016
@LindsayHill LindsayHill deleted the destroy_vm_changes branch November 2, 2018 00:04
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

1 participant