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 tests for hcloud remediation and hetzner baremetal remediation controller #1000

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

aniruddha2000
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes # https://github.com/syself/open-source/issues/76

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

@aniruddha2000 aniruddha2000 marked this pull request as ready for review October 9, 2023 06:22
@aniruddha2000 aniruddha2000 marked this pull request as draft October 9, 2023 09:19
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
controllers/hcloudremediation_controller_test.go Outdated Show resolved Hide resolved
@aniruddha2000 aniruddha2000 force-pushed the ani/issues/open-source/76 branch 3 times, most recently from 6f0fcd3 to d29991e Compare October 12, 2023 09:05
janiskemper
janiskemper previously approved these changes Oct 13, 2023
@aniruddha2000 aniruddha2000 marked this pull request as ready for review October 13, 2023 07:06
@syself-bot syself-bot bot added the area/code Changes made in the code directory label Oct 13, 2023
@syself-bot syself-bot bot added the size/L Denotes a PR that changes 200-800 lines, ignoring generated files. label Oct 13, 2023
@aniruddha2000
Copy link
Contributor Author

E2Es are failing, should we block the merge like other PRs until e2e fixes?

@aniruddha2000 aniruddha2000 force-pushed the ani/issues/open-source/76 branch 2 times, most recently from 7718238 to 8facc81 Compare October 14, 2023 19:15
@aniruddha2000 aniruddha2000 merged commit cb8f745 into main Oct 15, 2023
9 checks passed
@aniruddha2000 aniruddha2000 deleted the ani/issues/open-source/76 branch October 15, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/code Changes made in the code directory size/L Denotes a PR that changes 200-800 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants