-
Notifications
You must be signed in to change notification settings - Fork 32
♻️ Shorten traefik config placeholder svc name #8217
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
♻️ Shorten traefik config placeholder svc name #8217
Conversation
The name is service name is too long and causes >64 len char issue when deploying in prod. This fixes the problem by using a short name Related PR/s * original introduction of this servicehttps://github.com/ITISFoundation/pull/7899 * corresponding ops changes ITISFoundation/osparc-ops-environments#1177
|
@mergify queue |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8217 +/- ##
==========================================
+ Coverage 86.09% 88.08% +1.99%
==========================================
Files 1910 1910
Lines 73396 73396
Branches 1301 1301
==========================================
+ Hits 63191 64653 +1462
+ Misses 9817 8356 -1461
+ Partials 388 387 -1
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
🛑 Configuration not compatible with a branch protection settingThe branch protection setting |
…laceholder-service
|
🧪 CI InsightsHere's what we observed from your CI run for 8a32fd5. ✅ Passed Jobs With Interesting Signals
|
|
@matusdrobuliak66 please tag in v1.84.1 |



What do these changes do?
The name is service name is too long and causes >64 len char issue when deploying in prod. This fixes the problem by using a short name
To be hotfixed in v1.84.1
Related issue/s
Related PR/s
How to test
Dev-ops