Skip to content

Refactor Status Updater to be resource agnostic #1814

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

Merged
merged 10 commits into from
Apr 16, 2024
Merged

Refactor Status Updater to be resource agnostic #1814

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
6befd1e
Refactor Status Updater to be resource agnostic
pleshakov Apr 5, 2024
4e25bf9
Expect this ToNot(Panic())
pleshakov Apr 15, 2024
8380352
Move it to BeforeAll
pleshakov Apr 15, 2024
96b882b
"gateways" -> groupGateways
pleshakov Apr 15, 2024
2db2008
extract a function for all the status stuff
pleshakov Apr 15, 2024
4b318bb
Restore removed conditional GatewayClass update feature
pleshakov Apr 15, 2024
4f3c992
gatewayCtrlName -> gatewayCtlrName
pleshakov Apr 15, 2024
bed9821
eliminate the else block altogether
pleshakov Apr 16, 2024
d8a9b41
add more context to the test description
pleshakov Apr 16, 2024
196a991
Merge branch 'main' into techdebt/refactor-status-updater
pleshakov Apr 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'main' into techdebt/refactor-status-updater
  • Loading branch information
@pleshakov
pleshakov authored Apr 16, 2024
commit 196a991968159d34c71c6dbfebd32d115404c943

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.