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

fix(shulker-proxy-agent): crash when LoadBalancer service does not have any ingress in status #627

Merged
merged 3 commits into from
Aug 25, 2024

Conversation

jeremylvln
Copy link
Owner

@jeremylvln jeremylvln commented Aug 24, 2024

Also, now watch the Service's updates, so the external IP is updated directly without any restart.

Closes #626

@jeremylvln jeremylvln self-assigned this Aug 24, 2024
@jeremylvln jeremylvln added type:bug 🐛 Something isn't working priority:critical 🚒 Handle this issue as soon as possible labels Aug 24, 2024
Copy link

codecov bot commented Aug 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 31.69%. Comparing base (f13ab89) to head (0bb929f).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #627   +/-   ##
=======================================
  Coverage   31.69%   31.69%           
=======================================
  Files          58       58           
  Lines        3124     3124           
=======================================
  Hits          990      990           
  Misses       2134     2134           
Flag Coverage Δ
shulker-operator 59.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...or/src/reconcilers/minecraft_cluster/proxy_role.rs 75.00% <ø> (ø)

@jeremylvln jeremylvln force-pushed the fix/proxy-crash-lb-service branch from 1954aeb to 0bb929f Compare August 25, 2024 08:22
@jeremylvln jeremylvln enabled auto-merge (squash) August 25, 2024 08:26
@jeremylvln jeremylvln merged commit 8be044a into main Aug 25, 2024
10 checks passed
@jeremylvln jeremylvln deleted the fix/proxy-crash-lb-service branch August 25, 2024 08:28
jeremylvln pushed a commit that referenced this pull request Aug 25, 2024
## [0.10.0](v0.9.0...v0.10.0) (2024-08-25)

### ✨ Features ✨

* **shulker-proxy-agent:** expose player teleporting in API ([#629](#629)) ([111ca37](111ca37))
* support providing external servers ([#631](#631)) ([86b4804](86b4804))

### 🐛 Bug Fixes 🐛

* **shulker-proxy-agent:** crash when LoadBalancer service does not have any ingress in status ([#627](#627)) ([8be044a](8be044a))
* **shulker-proxy-agent:** do not watch service if preferred address is provided ([a8e2e70](a8e2e70))
Copy link

🎉 This issue has been resolved in version 0.10.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:critical 🚒 Handle this issue as soon as possible type:bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shulker Agent crashed, stopping proxy after server is added to proxy
1 participant