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

Add miq_task_id column to miq_queue table #167

Merged
merged 2 commits into from
Mar 15, 2018

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Feb 13, 2018

Linking miq_queue and miq_tasks table will:

blocking ManageIQ/manageiq#17015

\cc @Fryguy

…to the list of arguments when message deliver. Aslo, it will simplify activating task

https://bugzilla.redhat.com/show_bug.cgi?id=1543289
@miq-bot miq-bot added the wip label Feb 13, 2018
@yrudman yrudman changed the title [WIP] Link miq_queue and miq_tasks tables Add miq_task_id column to miq_queue table Mar 12, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Mar 12, 2018

@miq-bot remove-label wip

@miq-bot miq-bot removed the wip label Mar 12, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Mar 12, 2018

@miq-bot add-label core, sql migration

@miq-bot
Copy link
Member

miq-bot commented Mar 12, 2018

@yrudman Cannot apply the following labels because they are not recognized: core, sql migration

@yrudman
Copy link
Contributor Author

yrudman commented Mar 13, 2018

@miq-bot assign @Fryguy

@@ -0,0 +1,5 @@
class LinkQueueAndTask < ActiveRecord::Migration[5.0]
def change
add_reference :miq_queue, :miq_task, :type => "bigint", :index => true
Copy link
Member

Choose a reason for hiding this comment

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

You shouldn't need the explicit :type => "bigint", but I could be wrong. Even so, we should probably use a sybol there instead of a string. Also, not sure on adding an index. @kbrock Thoughts on this index?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my mistake, we do not need index now.

@miq-bot
Copy link
Member

miq-bot commented Mar 14, 2018

Checked commits yrudman/manageiq-schema@7aefb23~...0ae2566 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 1 offense detected

db/migrate/20180213204846_link_queue_and_task.rb

@Fryguy Fryguy merged commit 076ae3d into ManageIQ:master Mar 15, 2018
@Fryguy Fryguy added this to the Sprint 82 Ending Mar 26, 2018 milestone Mar 15, 2018
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