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.

Sign up for GitHub

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

Jump to bottom

Update the e2e tests to reflect the new defaults #2222

Merged
merged 5 commits into from
Feb 25, 2025
Merged

Update the e2e tests to reflect the new defaults #2222

merged 5 commits into from
Feb 25, 2025

Conversation

johscheuer
Copy link
Member

Description

Update the e2e tests to reflect the new defaults.

Type of change

  • Other

Discussion

Make sure we use the same defaults in the e2e tests as we use in the operator.

Testing

CI will be running e2e tests.

Documentation

Follow-up

@foundationdb-ci
Copy link

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

  • Commit ID: dc3ed87
  • Duration 3:21:34
  • 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 force-pushed the update-e2e-new-defaults branch from dc3ed87 to 1bada28 Compare February 24, 2025 12:44
@johscheuer johscheuer marked this pull request as ready for review February 24, 2025 13:23
@foundationdb-ci
Copy link

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

  • Commit ID: 1bada28
  • Duration 3:16:56
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@johscheuer johscheuer force-pushed the update-e2e-new-defaults branch from 1bada28 to cd802df Compare February 24, 2025 16:22
@johscheuer johscheuer added the bug Something isn't working label Feb 24, 2025
@foundationdb-ci
Copy link

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

  • Commit ID: cd802df
  • Duration 2:52:09
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • 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 25, 2025
@johscheuer johscheuer reopened this Feb 25, 2025
@foundationdb-ci
Copy link

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

  • Commit ID: cd802df
  • Duration 2:49:49
  • 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
Copy link
Member Author

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

  • Commit ID: cd802df
  • Duration 2:49:49
  • 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)
• [FAILED] [352.979 seconds]
Operator HA tests when deleting all Pods in the primary [It] should change the coordinators [e2e, pr]
/codebuild/output/src2553396698/src/github.com/FoundationDB/fdb-kubernetes-operator/e2e/test_operator_ha/operator_ha_test.go:132

  [FAILED] Timed out after 300.001s.
  Expected
      <string>: operator_ha_k3nbdivx_primary:m2lDQWzTtKLa4vtNhV9YApkFuhWuxfOZ@operator-ha-k3nbdivx-primary-log-25547.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-log-65948.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-log-72905.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-sat-log-60652.operator-ha-k3nbdivx-primary-sat.pr-1229-operator-ha-287nlb9x-primary-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-sat-log-70419.operator-ha-k3nbdivx-primary-sat.pr-1229-operator-ha-287nlb9x-primary-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-sat-log-13376.operator-ha-k3nbdivx-remote-sat.pr-1229-operator-ha-287nlb9x-remote-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-sat-log-95432.operator-ha-k3nbdivx-remote-sat.pr-1229-operator-ha-287nlb9x-remote-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-storage-16229.operator-ha-k3nbdivx-remote.pr-1229-operator-ha-287nlb9x-remote.svc.cluster.local:4502:tls,operator-ha-k3nbdivx-remote-storage-76474.operator-ha-k3nbdivx-remote.pr-1229-operator-ha-287nlb9x-remote.svc.cluster.local:4502:tls
  not to equal
      <string>: operator_ha_k3nbdivx_primary:m2lDQWzTtKLa4vtNhV9YApkFuhWuxfOZ@operator-ha-k3nbdivx-primary-log-25547.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-log-65948.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-log-72905.operator-ha-k3nbdivx-primary.pr-1229-operator-ha-287nlb9x-primary.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-sat-log-60652.operator-ha-k3nbdivx-primary-sat.pr-1229-operator-ha-287nlb9x-primary-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-primary-sat-log-70419.operator-ha-k3nbdivx-primary-sat.pr-1229-operator-ha-287nlb9x-primary-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-sat-log-13376.operator-ha-k3nbdivx-remote-sat.pr-1229-operator-ha-287nlb9x-remote-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-sat-log-95432.operator-ha-k3nbdivx-remote-sat.pr-1229-operator-ha-287nlb9x-remote-sat.svc.cluster.local:4500:tls,operator-ha-k3nbdivx-remote-storage-16229.operator-ha-k3nbdivx-remote.pr-1229-operator-ha-287nlb9x-remote.svc.cluster.local:4502:tls,operator-ha-k3nbdivx-remote-storage-76474.operator-ha-k3nbdivx-remote.pr-1229-operator-ha-287nlb9x-remote.svc.cluster.local:4502:tls
  In [It] at: /codebuild/output/src2553396698/src/github.com/FoundationDB/fdb-kubernetes-operator/e2e/test_operator_ha/operator_ha_test.go:156 @ 02/25/25 06:25:23.24

I double check this failure, but seems like the pods where created fast enough before the coordinators were changed.

nicmorales9
nicmorales9 approved these changes Feb 25, 2025
View reviewed changes
Copy link
Contributor

@nicmorales9 nicmorales9 left a comment

Choose a reason for hiding this comment

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

some minor questions but overall looks good

@foundationdb-ci
Copy link

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

  • Commit ID: f7840f1
  • Duration 3:03:22
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@johscheuer johscheuer merged commit bf8922c into FoundationDB:main Feb 25, 2025
7 checks passed
@johscheuer johscheuer deleted the update-e2e-new-defaults branch February 25, 2025 15:24
@foundationdb-ci
Copy link

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

  • Commit ID: 07ac1be
  • Duration 2:46:37
  • 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)

@foundationdb-ci
Copy link

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

  • Commit ID: 40a8ee0
  • Duration 2:45:10
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • 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
Reviewers

@nicmorales9 nicmorales9 nicmorales9 approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@johscheuer @foundationdb-ci @nicmorales9