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 API request scheme for preemptible flag when provisioning #70

Merged
merged 1 commit into from
Aug 28, 2018

Conversation

tumido
Copy link
Member

@tumido tumido commented Aug 28, 2018

Adapt to changed API request scheme, so the :preemptible flag is sent as:

{
  ...
  "scheduling": {
    "preemptible": [true,false]
    ...
  }
  ...
}

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1619298

@tumido
Copy link
Member Author

tumido commented Aug 28, 2018

@miq-bot add_label bug

@miq-bot miq-bot added the bug label Aug 28, 2018
@tumido tumido force-pushed the preemptible_fixed branch from bdc02f2 to e64609f Compare August 28, 2018 10:40
@tumido
Copy link
Member Author

tumido commented Aug 28, 2018

Verified in GCE that it indeed creates a preemptible instance now.

@miq-bot
Copy link
Member

miq-bot commented Aug 28, 2018

Checked commit tumido@e64609f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@tumido
Copy link
Member Author

tumido commented Aug 28, 2018

cc @bronaghs, can you please review this? 😉

@bronaghs bronaghs merged commit 364518e into ManageIQ:master Aug 28, 2018
@bronaghs bronaghs added this to the Sprint 94 Ending Sept 10, 2018 milestone Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants