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

*: Support display the raft status of the node in nodes.conditions. #284 #285

Merged
merged 4 commits into from
Nov 16, 2021

Conversation

runkecheng
Copy link
Collaborator

What type of PR is this?

/enhancement

Which issue(s) this PR fixes?

Fixes ##284

What this PR does?

Summary:

  • Adding the basic framework code of the xenon http request.
  • Support raft/status and raft/trytoleader
  • Support display the raft status of the node in nodes.conditions.

Special notes for your reviewer?

@runkecheng runkecheng added the enhancement New feature or request label Nov 3, 2021
@runkecheng runkecheng added this to the v2.1.1 milestone Nov 3, 2021
@runkecheng runkecheng self-assigned this Nov 3, 2021
@runkecheng
Copy link
Collaborator Author

runkecheng commented Nov 3, 2021

Need a short little alias for internal .

/home/runner/work/radondb-mysql-kubernetes/radondb-mysql-kubernetes/mysqlcluster/syncer/statefulset.go
  (41, 2)  ST1001  should not use dot imports

@runkecheng
Copy link
Collaborator Author

After modification:

  Nodes:
    Conditions:
      Last Transition Time:  2021-11-04T09:53:05Z
      Status:                False
      Type:                  Lagged
      Last Transition Time:  2021-11-04T09:53:05Z
      Status:                True
      Type:                  Leader
      Last Transition Time:  2021-11-04T09:53:05Z
      Status:                False
      Type:                  ReadOnly
      Last Transition Time:  2021-11-04T09:53:05Z
      Status:                False
      Type:                  Replicating
    Name:                    sample-mysql-0.sample-mysql.default
    Raft Status:
      Leader:  sample-mysql-0.sample-mysql.default:8801
      Nodes:
        sample-mysql-0.sample-mysql.default:8801
        sample-mysql-1.sample-mysql.default:8801
        sample-mysql-2.sample-mysql.default:8801
      Role:  LEADER
    Conditions:
      Last Transition Time:  2021-11-04T09:55:25Z
      Status:                False
      Type:                  Lagged
      Last Transition Time:  2021-11-04T09:55:25Z
      Status:                False
      Type:                  Leader
      Last Transition Time:  2021-11-04T09:55:25Z
      Status:                True
      Type:                  ReadOnly
      Last Transition Time:  2021-11-04T09:55:25Z
      Status:                True
      Type:                  Replicating
    Name:                    sample-mysql-1.sample-mysql.default
    Raft Status:
      Leader:  sample-mysql-0.sample-mysql.default:8801
      Nodes:
        sample-mysql-0.sample-mysql.default:8801
        sample-mysql-1.sample-mysql.default:8801
        sample-mysql-2.sample-mysql.default:8801
      Role:  FOLLOWER
    Conditions:
      Last Transition Time:  2021-11-04T09:58:25Z
      Status:                False
      Type:                  Lagged
      Last Transition Time:  2021-11-04T09:58:25Z
      Status:                False
      Type:                  Leader
      Last Transition Time:  2021-11-04T09:58:25Z
      Status:                True
      Type:                  ReadOnly
      Last Transition Time:  2021-11-04T09:58:25Z
      Status:                True
      Type:                  Replicating
    Name:                    sample-mysql-2.sample-mysql.default
    Raft Status:
      Leader:  sample-mysql-0.sample-mysql.default:8801
      Nodes:
        sample-mysql-0.sample-mysql.default:8801
        sample-mysql-1.sample-mysql.default:8801
        sample-mysql-2.sample-mysql.default:8801
      Role:     FOLLOWER

@runkecheng runkecheng marked this pull request as ready for review November 4, 2021 10:06
@runkecheng
Copy link
Collaborator Author

runkecheng commented Nov 5, 2021

Need a short little alias for internal .

/home/runner/work/radondb-mysql-kubernetes/radondb-mysql-kubernetes/mysqlcluster/syncer/statefulset.go
  (41, 2)  ST1001  should not use dot imports

ping @andyli029 @acekingke I change it back to internal.

@runkecheng runkecheng force-pushed the xenon_http_executor branch 3 times, most recently from 90de1f1 to 4144fe6 Compare November 5, 2021 06:23
internal/http_executor.go Outdated Show resolved Hide resolved
internal/xenon_request.go Outdated Show resolved Hide resolved
internal/xenon_request.go Outdated Show resolved Hide resolved
mysqlcluster/syncer/statefulset.go Show resolved Hide resolved
mysqlcluster/syncer/statefulset.go Show resolved Hide resolved
@acekingke
Copy link
Contributor

LGTM

@runkecheng runkecheng force-pushed the xenon_http_executor branch 2 times, most recently from b5a0771 to 50fd4be Compare November 11, 2021 08:29
@andyli029
Copy link
Contributor

LGTM.

@andyli029 andyli029 merged commit be1a9fa into radondb:main Nov 16, 2021
zhl003 pushed a commit to zhl003/radondb-mysql-kubernetes that referenced this pull request Aug 17, 2022
*: Support display the raft status of the node in nodes.conditions. radondb#284
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants