-
Notifications
You must be signed in to change notification settings - Fork 125
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
Add user_id,group_id,tenant_id to MiqQueue #83
Conversation
92dd842
to
ad628bb
Compare
db/schema.yml
Outdated
- type | ||
- object_labels |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not happen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the migrations were run out of order?
@kbrock Please review. |
Checked commits mkanoor/manageiq-schema@ad628bb~...3c70428 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we had unlimited time, sure would have liked to see these rpc/background calls get converted to events.
But yes, this is missing functionality in the queue. Automate currently does this manually. Will be nice to have this as a first class construct.
This is needed so that the Queue can run commands in the context of the user, group and tenant.