Skip to content

web/vtctld: Hook up actions in new interface.#885

Merged
enisoc merged 1 commit intovitessio:masterfrom
enisoc:vtctld-actions
Jul 16, 2015
Merged

web/vtctld: Hook up actions in new interface.#885
enisoc merged 1 commit intovitessio:masterfrom
enisoc:vtctld-actions

Conversation

@enisoc
Copy link
Copy Markdown
Member

@enisoc enisoc commented Jul 15, 2015

@alainjobart
Copy link
Copy Markdown
Contributor

LGTM

enisoc added a commit that referenced this pull request Jul 16, 2015
web/vtctld: Hook up actions in new interface.
@enisoc enisoc merged commit 7f4856f into vitessio:master Jul 16, 2015
@enisoc enisoc deleted the vtctld-actions branch July 16, 2015 00:32
rsajwani pushed a commit to planetscale/vitess that referenced this pull request Aug 1, 2022
…) (vitessio#885)

* Use introduced tablet manager RPCs in VTOrc (vitessio#10467)

* feat: use the new RPC for resetting replication

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: add code to get the full status and use it

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: remove uptime from instance propoerties in VTOrc

Signed-off-by: Manan Gupta <manan@planetscale.com>

* test: add end to end test for checking the output of ReadInstanceBufferable

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: remove a mysql query for server id, etc and use full status information instead

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: use primary status output instead of mysql query

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: remove semi-sync available, since it is unused

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: use semi sync enable from full status instead of mysql query

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: use semi sync statuses from full status instead of mysql query

Signed-off-by: Manan Gupta <manan@planetscale.com>

* test: check output of more fields in the test

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: remove ReplicationCredentialsAvailable since it is unused

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: use executed, purged gtid set, server uuid, gtid mode and binlog row image from full status instead of mysql query

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: add testing for replication status fields

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: check for nil values to prevent panics when some fields are empty

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: use replication status from full status instead of mysql query

Signed-off-by: Manan Gupta <manan@planetscale.com>

* test: add license and shorten package name

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat: fix test flakiness

Signed-off-by: Manan Gupta <manan@planetscale.com>

* feat:fix test to work with mysql 8.0 too

Signed-off-by: Manan Gupta <manan@planetscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants