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-operator): do not let Agones map the server ports #330

Merged
merged 1 commit into from
Jan 8, 2024

Conversation

jeremylvln
Copy link
Owner

When Agones maps the ports of the GameServer, it also binds a host port. It's not something we want for the servers as the network is routed through the proxies, so internally of the Kubernetes cluster.

@jeremylvln jeremylvln self-assigned this Jan 8, 2024
@jeremylvln jeremylvln added the type:bug 🐛 Something isn't working label Jan 8, 2024
@jeremylvln jeremylvln enabled auto-merge (squash) January 8, 2024 10:17
Copy link

codecov bot commented Jan 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (43a56b1) 30.81% compared to head (a2536ed) 30.79%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
- Coverage   30.81%   30.79%   -0.03%     
==========================================
  Files          57       57              
  Lines        3115     3114       -1     
==========================================
- Hits          960      959       -1     
  Misses       2155     2155              
Flag Coverage Δ
shulker-operator 58.03% <ø> (-0.03%) ⬇️

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

Files Coverage Δ
...tor/src/reconcilers/minecraft_server/gameserver.rs 90.45% <ø> (-0.04%) ⬇️

@jeremylvln jeremylvln merged commit d1a9955 into main Jan 8, 2024
10 checks passed
@jeremylvln jeremylvln deleted the fix/do-not-expose-server-ports branch January 8, 2024 10:22
jeremylvln pushed a commit that referenced this pull request Jan 8, 2024
## [0.5.1](v0.5.0...v0.5.1) (2024-01-08)

### 🐛 Bug Fixes 🐛

* **deps:** update dependency io.grpc:grpc-netty-shaded to v1.60.1 ([#322](#322)) ([8303b6e](8303b6e))
* **deps:** update dependency net.kyori:adventure-platform-bungeecord to v4.3.2 ([#326](#326)) ([a9d6e34](a9d6e34))
* **deps:** update grpc to v1.60.1 ([#323](#323)) ([87e8641](87e8641))
* **shulker-operator:** do not let Agones map the server ports ([#330](#330)) ([d1a9955](d1a9955))

### 📚 Documentation 📚

* promote next to latest ([8178611](8178611))
Copy link

github-actions bot commented Jan 8, 2024

🎉 This PR is included in version 0.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

jeremylvln added a commit that referenced this pull request Jan 12, 2024
jeremylvln pushed a commit that referenced this pull request Jan 12, 2024
## [0.5.2](v0.5.1...v0.5.2) (2024-01-12)

### ⚠️ Reverts ⚠️

* Revert "fix(shulker-operator): do not let Agones map the server ports (#330)" ([5a376cf](5a376cf)), closes [#330](#330)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant