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 a #backlog method to PglogicalSubscription objects #14010

Merged

Conversation

carbonin
Copy link
Member

This will show the replication backlog in bytes for the particular subscription.

This uses the functionality added to the pg-pglogical gem in ManageIQ/pg-pglogical#8

This will also need a new gem version to pull in the changes so I will mark this as WIP until that is worked out.

@carbonin carbonin requested a review from Fryguy February 21, 2017 21:36
@carbonin carbonin added the wip label Feb 21, 2017
carbonin added a commit to carbonin/manageiq-ui-classic that referenced this pull request Feb 21, 2017
This is the UI for the new value exposed for the subscription in
ManageIQ/manageiq#14010
This will show the replication backlog in bytes for the particular
subscription.

This uses the functionality added to the pg-pglogical gem in
ManageIQ/pg-pglogical#8
@carbonin carbonin force-pushed the add_replication_backlog_to_subscription branch from b9e6173 to 0f6760f Compare February 22, 2017 21:36
@carbonin carbonin closed this Feb 23, 2017
@carbonin carbonin reopened this Feb 23, 2017
@carbonin carbonin changed the title [WIP] Add a #backlog method to PglogicalSubscription objects Add a #backlog method to PglogicalSubscription objects Feb 27, 2017
@carbonin carbonin removed the wip label Feb 27, 2017
@miq-bot
Copy link
Member

miq-bot commented Feb 27, 2017

Checked commits carbonin/manageiq@7327527~...d178ba4 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 0 offenses detected
Everything looks good. 👍

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.

👍

@gtanzillo gtanzillo added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 27, 2017
@gtanzillo gtanzillo merged commit c363053 into ManageIQ:master Feb 27, 2017
@gtanzillo gtanzillo mentioned this pull request Mar 1, 2017
@carbonin carbonin deleted the add_replication_backlog_to_subscription branch March 7, 2017 16:31
TomasKohoutek pushed a commit to TomasKohoutek/manageiq-ui-classic that referenced this pull request Apr 7, 2017
This is the UI for the new value exposed for the subscription in
ManageIQ/manageiq#14010
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