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 replication set-up methods to be queued on UI side #17956

Merged
merged 5 commits into from
Sep 12, 2018

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Sep 7, 2018

Add replication set-up methods to be queued on UI side

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1562956

Corresponding PR on UI side: ManageIQ/manageiq-ui-classic#4628

@miq-bot add-label core

@miq-bot

This comment has been minimized.

@miq-bot miq-bot added the wip label Sep 7, 2018
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch 4 times, most recently from d2955c2 to db57156 Compare September 7, 2018 16:26
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch from 98395cb to 16bc242 Compare September 7, 2018 18:05
@miq-bot
Copy link
Member

miq-bot commented Sep 7, 2018

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

@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch from 16bc242 to 59ac0a2 Compare September 10, 2018 11:33
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch 3 times, most recently from e820150 to e3a7da4 Compare September 11, 2018 20:04
@yrudman yrudman changed the title [WIP] Queue operations for pglogical replication set-up Add replication set-up methods to be queued on UI side Sep 11, 2018
@miq-bot miq-bot removed the wip label Sep 11, 2018
@yrudman yrudman closed this Sep 12, 2018
@yrudman yrudman reopened this Sep 12, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Sep 12, 2018

@carbonin could you review

lib/miq_pglogical.rb Outdated Show resolved Hide resolved
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch from 4303fd1 to 40672f4 Compare September 12, 2018 13:41
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch 3 times, most recently from e50b801 to 7b6b2f5 Compare September 12, 2018 15:42
@yrudman yrudman force-pushed the queue-pglogical-replication-setup branch from 7b6b2f5 to 0cfba4c Compare September 12, 2018 15:54
@miq-bot
Copy link
Member

miq-bot commented Sep 12, 2018

Checked commits yrudman/manageiq@e7c0712~...0cfba4c with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 0 offenses detected
Everything looks fine. 🍰

@carbonin carbonin self-assigned this Sep 12, 2018
@carbonin carbonin added the bug label Sep 12, 2018
Copy link
Member

@carbonin carbonin left a comment

Choose a reason for hiding this comment

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

Thanks for picking this up @yrudman

@carbonin
Copy link
Member

Holding off on assigning the Gaprindashvili label for now until I can get feedback from @dmetzger57 in the BZ.

@carbonin
Copy link
Member

Merging while red because of known sporadic spec failures which are only present in one version of ruby tested in this PR.

@carbonin carbonin merged commit 520afad into ManageIQ:master Sep 12, 2018
@carbonin carbonin added this to the Sprint 95 Ending Sep 24, 2018 milestone Sep 12, 2018
@yrudman yrudman deleted the queue-pglogical-replication-setup branch September 12, 2018 18:09
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