Skip to content
This repository has been archived by the owner on Jul 9, 2020. It is now read-only.

Add remote execution jobs to community-templates #268

Merged
merged 1 commit into from Apr 28, 2016

Conversation

stbenjam
Copy link
Member

No description provided.


## Contributing

If you think a job is useful to include here, feel free to open an PR. Newer
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we name a specific version that started to support this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I'm just not 100% sure of the version number yet.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's not released yet then this is fine for now. We'll fix it later.

@mmoll
Copy link
Contributor

mmoll commented Apr 22, 2016

As I'm not really following r_ex development at the moment: Is theforeman/foreman_templates#23 required for this to be really useful?

@stbenjam
Copy link
Member Author

As I'm not really following r_ex development at the moment: Is theforeman/foreman_templates#23 required for this to be really useful?

I don't think so, we definitely want that in foreman_templates, but this was more to encourage contributions.

More than a few people have asked where they could contribute additional templates/submit PR's, and it's a lot easier if it's all in the community-templates repo.

foreman_remote_execution requires redmine tickets, and just overall a higher bar to entry.

@stbenjam
Copy link
Member Author

Also if you are all OK with this change, it'd be good if some of the REX people had commit here. I don't at least, not sure about @ares. But I suppose we can deal with that when we actually get a PR from someone.

@mmoll
Copy link
Contributor

mmoll commented Apr 22, 2016

@ares does have push 👐 permissions here.
I'd say this is OK to merge once r_ex nightlies can export jobs and I would be highly fond of also having the import functionality in foreman_templates.

@ekohl
Copy link
Member

ekohl commented Apr 23, 2016

I'd be fine with giving @stbenjam merge permissions as well.

@mmoll
Copy link
Contributor

mmoll commented Apr 23, 2016

Given the Puppet work, @stbenjam is doing over at Katello, I'd also have no objections adding him the the installer team here. 👍

@stbenjam
Copy link
Member Author

Updated, 0.3.2 will be released soon (package PR's are open) and includes export/import.

If this is good to go, I'd appreciate a merge before the demo this morning. It's not support important, though.

@mmoll mmoll merged commit bd65953 into theforeman:develop Apr 28, 2016
@stbenjam stbenjam deleted the rex branch April 28, 2016 11:34
@mmoll
Copy link
Contributor

mmoll commented Apr 28, 2016

Sure thing, merged, thanks @stbenjam! 🍦

derektamsen pushed a commit to squarit/community-templates that referenced this pull request Jan 18, 2017
ares pushed a commit to ares/community-templates that referenced this pull request Aug 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants