-
Notifications
You must be signed in to change notification settings - Fork 89
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
Document how to change port for alarm logger #3107
Comments
I found #2277 (comment) recently, but as far as I know the |
Yes, SpringBoot adds too many dependencies and obfuscates the configuration. We should remove it and have all services use a plain "main()" like the alarm server, scan server, RDB archive engine |
Umm... There are quite a few other benefits of springboot, we are trying to merge 2 startup and configuration processes here which has resulted in some issues. I will look into cleaning it up. |
Nah, it's a complex framework just like Eclipse was. It expects you to know which annotations to add and where to place files without a way to follow breadcrumbs or call trees. In ~4 years there'll be a springboot update that forces you re-do everything. |
We should make all services Spring Boot-based. |
Not long ago, I fell into a rabbit hole on why my
server.port
property was not recognized by Phoebus alarm logger.After a while, I found out that it's because the
-properties /path/to/config.properties
argument doesn't forward the loaded properties to SpringBoot. So my solution was to also specify the environment variableSPRING_CONFIG_IMPORT=/path/to/config.properties
.It would be nice if there was an official, documented way of doing this, as port 8080 is used by quite a number of EPICS-related services by default.
The text was updated successfully, but these errors were encountered: