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

Drop 'jobs.agent_id' column. #14634

Merged
merged 1 commit into from
May 12, 2017

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Apr 4, 2017

blocked:

All references to 'jobs.agent_id' were removed in #14030, and this column not in use.
Dropping it will make future merging of Job and MiqTask models simpler.

@miq-bot add-label core, technical debt, wip

@yrudman yrudman changed the title Drop 'jobs.jobs.agent_id' column. We can do it after all references to 'Jo… [WIP] Drop 'jobs.agent_id' column. We can do it after all references to 'Jo… Apr 4, 2017
@miq-bot miq-bot changed the title [WIP] Drop 'jobs.agent_id' column. We can do it after all references to 'Jo… [WIP] Drop 'jobs.jobs.agent_id' column. We can do it after all references to 'Jo… Apr 4, 2017
@yrudman yrudman changed the title [WIP] Drop 'jobs.jobs.agent_id' column. We can do it after all references to 'Jo… [WIP] Drop 'jobs.jobs.agent_id' column. Apr 4, 2017
@yrudman yrudman changed the title [WIP] Drop 'jobs.jobs.agent_id' column. [WIP] Drop 'jobs.agent_id' column. Apr 4, 2017
@yrudman yrudman force-pushed the drop-agent_id_column-from-jobs branch 2 times, most recently from 709eee4 to d598c7e Compare April 4, 2017 16:28
@yrudman yrudman changed the title [WIP] Drop 'jobs.agent_id' column. Drop 'jobs.agent_id' column. Apr 5, 2017
@miq-bot miq-bot removed the wip label Apr 5, 2017
@yrudman
Copy link
Contributor Author

yrudman commented Apr 6, 2017

@miq-bot add-label fine/no

@miq-bot miq-bot added the fine/no label Apr 6, 2017
Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

👍

@miq-bot
Copy link
Member

miq-bot commented May 9, 2017

This pull request is not mergeable. Please rebase and repush.

Copy link
Member

@Fryguy Fryguy left a comment

Choose a reason for hiding this comment

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

LGTM...It just got unmergeable because of the PR that deleted the process column

@yrudman yrudman force-pushed the drop-agent_id_column-from-jobs branch from d598c7e to 9144263 Compare May 9, 2017 21:33
@yrudman yrudman force-pushed the drop-agent_id_column-from-jobs branch from b6345c5 to f982628 Compare May 9, 2017 23:09
@yrudman yrudman force-pushed the drop-agent_id_column-from-jobs branch from f982628 to 404f3e8 Compare May 10, 2017 19:30
@miq-bot
Copy link
Member

miq-bot commented May 10, 2017

Checked commit yrudman@404f3e8 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@yrudman
Copy link
Contributor Author

yrudman commented May 10, 2017

@Fryguy it is mergeable now

@gtanzillo gtanzillo added this to the Sprint 61 Ending May 22, 2017 milestone May 12, 2017
@gtanzillo gtanzillo merged commit 1d7cc86 into ManageIQ:master May 12, 2017
@yrudman yrudman deleted the drop-agent_id_column-from-jobs branch May 12, 2017 19:46
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.

4 participants