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

Sample actions for tasks development #392

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

iNecas
Copy link
Member

@iNecas iNecas commented Mar 4, 2019

A set of actions with pre-defined behaviour for easier simulation of
various tasks states: these actions are development only.

A set of actions with pre-defined behaviour for easier simulation of
various tasks states: these actions are development only.
@theforeman-bot
Copy link
Member

There were the following issues with the commit message:

  • 93d17bf must be in the format fixes #redmine_number - brief description

If you don't have a ticket number, please create an issue in Redmine.

More guidelines are available in Coding Standards or on the Foreman wiki.


This message was auto-generated by Foreman's prprocessor

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

LGTM, one comment inline, but I don't feel too strongly about it

#
# # Task that fails but resumes (ending in stopped state/warning result)
# ForemanTasks.sync_task(Actions::Try::Skip)
unless Rails.env.production?
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd move this one level higher so we won't end up with empty Actions::Try module in production environments

@adamruzicka
Copy link
Contributor

@ezr-ondrej opinions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants