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

Pass UUID app configuration in during server startup #79

Closed
kburke opened this issue May 17, 2022 · 1 comment
Closed

Pass UUID app configuration in during server startup #79

kburke opened this issue May 17, 2022 · 1 comment
Assignees
Labels
P Pitt dev team

Comments

@kburke
Copy link

kburke commented May 17, 2022

Both app.py and uuid_worker.py have app.config.from_pyfile() statements, and are now configured to work with 'app.cfg' for HuBMAP and 'sennet.app.cfg' for SenNet. Currently there are problems if these two statements are not coordinated to use the same file.

Pass the configuration read by app.py from the file into app.config to the UUIDWorker() init() Change UUIDWork.init() to set global variables from the configuration parameters read by app.py, and not re-read the configuration file.

@shirey shirey added the P Pitt dev team label May 17, 2022
@shirey shirey changed the title Pass UUID app configuration in from server startup Pass UUID app configuration in during server startup May 31, 2022
@kburke
Copy link
Author

kburke commented Jun 9, 2022

Changing the description of this issue

FROM:
"Pass the filename to be used for configuration in from the server during startup for app.config.from_pyfile() execution in app.py, and to pass to the UUIDWorker() constructor during instantiation of worker in init(). Change UUIDWork.init() to use the argument for its load of app.config.from_pyfile()."

TO:
"Pass the configuration read by app.py from the file into app.config to the UUIDWorker() init() Change UUIDWork.init() to set global variables from the configuration parameters read by app.py, and not re-read the configuration file."

kburke pushed a commit that referenced this issue Jun 10, 2022
…ters to worker for it to use what it needs. Issue #80 Change Globus groups file override indication to be indicated by a file name.
yuanzhou added a commit that referenced this issue Jun 14, 2022
Issue #79 Read app configuration file once in app.py, and pass parame…
@shirey shirey closed this as completed Jul 12, 2022
shirey pushed a commit that referenced this issue Mar 9, 2023
…ters to worker for it to use what it needs. Issue #80 Change Globus groups file override indication to be indicated by a file name.
@shirey shirey added this to Pitt HIVE Jun 7, 2024
@shirey shirey moved this to Done in Pitt HIVE Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P Pitt dev team
Projects
None yet
Development

No branches or pull requests

2 participants