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 test case to validate that the upgrade is stuck when the new binary is missing #1558

Merged

Conversation

johscheuer
Copy link
Member

Description

This test makes sure that our upgrade is stuck when the new binary is missing from one of the Pods.

Type of change

  • Other

Discussion

Testing

Ran the test manually:

2023/03/29 07:44:44 storage-2 IncorrectCommandLine
2023/03/29 07:45:13 finished all tests, start deleting namespace jscheuermann-xtfzxidt
• [288.991 seconds]
------------------------------
[AfterSuite]
/Users/jscheuermann/go/src/github.com/FoundationDB/fdb-kubernetes-operator/e2e/test_operator_upgrades/operator_upgrades_test.go:53
[AfterSuite] PASSED [0.000 seconds]
------------------------------
[ReportAfterSuite] Autogenerated ReportAfterSuite for --junit-report
autogenerated by Ginkgo
[ReportAfterSuite] PASSED [0.004 seconds]
------------------------------

Ran 1 of 12 Specs in 288.991 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 11 Skipped
PASS | FOCUSED
FAIL    github.com/FoundationDB/fdb-kubernetes-operator/e2e/test_operator_upgrades      289.387s

Documentation

Added documentation in the test code.

Follow-up

@foundationdb-ci
Copy link

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

  • Commit ID: 55bebea
  • Duration 2:39:21
  • 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 merged commit 636ea72 into FoundationDB:main Mar 31, 2023
@johscheuer johscheuer deleted the add-test-case-stuck-missing-binary branch March 31, 2023 04:33
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.

3 participants