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

Update cluster file usage. #2230

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

johscheuer
Copy link
Member

Description

Make use of dedicated cluster file for the fdb client lib and ensure to reuse it. The cli tooling will get a temp cluster file which will be deleted after the command was used.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Discussion

Are there any design details that you would like to discuss further?

Testing

Ran e2e tests related to HA clusters.

I have to add another test in the e2e setup to see what happens when an operator is partitioned and all coordinators change (and no old coordinator is around) and how to resolve this case, in the previous setup a user could just update the SeedConnectionString.

Documentation

Will be updated before merging.

Follow-up

We should add a command in the kubectl plugin to update the connection string in the FoundationDBCluster status.

…to reuse it. The cli tooling will get a temp cluster file which will be deleted after the command was used
@foundationdb-ci
Copy link

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

  • Commit ID: 7341d9b
  • Duration 2:49:08
  • Result: ❌ FAILED
  • Error: Error while executing command: if $fail_test; then exit 1; fi. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@johscheuer johscheuer closed this Feb 28, 2025
@johscheuer johscheuer reopened this Feb 28, 2025
@foundationdb-ci
Copy link

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

  • Commit ID: 7341d9b
  • Duration 2:52:03
  • Result: ❌ FAILED
  • Error: Error while executing command: if $fail_test; then exit 1; fi. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

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