Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Add SystemJobTemplate#launch and the SystemJob class #100

Merged

Conversation

carbonin
Copy link
Contributor

@carbonin carbonin commented Apr 4, 2018

This will allow us to trigger a system job template manually and return the created system job.

@bdunne please review

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

@gmcculloug gmcculloug self-assigned this Apr 4, 2018
@gmcculloug gmcculloug requested a review from bdunne April 4, 2018 15:57
@gmcculloug
Copy link
Contributor

cc @syncrou

This will allow us to trigger a system job template manually
and return the created system job.

https://bugzilla.redhat.com/show_bug.cgi?id=1560478
@carbonin carbonin force-pushed the add_launch_for_system_job_templates branch from 7ed2a10 to f8c63b1 Compare April 4, 2018 18:04
@miq-bot
Copy link
Collaborator

miq-bot commented Apr 4, 2018

Some comments on commit carbonin@f8c63b1

spec/system_job_template_spec.rb

  • ⚠️ - 12 - Detected expect_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.

@miq-bot
Copy link
Collaborator

miq-bot commented Apr 4, 2018

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

@bdunne bdunne merged commit 245f44f into ansible:master Apr 4, 2018
@bdunne bdunne assigned bdunne and unassigned gmcculloug Apr 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants