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

dirac-install-tornado-service overwrites Port? #7126

Closed
sfayer opened this issue Jul 26, 2023 · 4 comments · Fixed by #7130
Closed

dirac-install-tornado-service overwrites Port? #7126

sfayer opened this issue Jul 26, 2023 · 4 comments · Fixed by #7130
Assignees
Milestone

Comments

@sfayer
Copy link
Member

sfayer commented Jul 26, 2023

Hi,

We're trying to use dirac-install-tornado-service Framework/TokenManager to install just the new endpoint on a machine that already has the base Tornado service installed and running... We've found that it overwrites the /Systems/Tornado/<Setup>/Port setting to 8443 even though we already had it set to 8444 (which then caused a failure as our Webapp is on 8443). Is this possibly a bug?

We also normally do service installs from a UI rather than directly on the server (as we don't have a dirac_admin user proxy readily available on there): We found that while it did install the TokenManager, it put the client hostname in the URLs section (with port 8444, despite overwriting it elsewhere) rather than the DIRAC server hostname.

Regards,
Simon

@fstagni
Copy link
Contributor

fstagni commented Jul 26, 2023

Is this with v8.0 or v8.1? (there are changes in this area)

@marianne013
Copy link
Contributor

v8.0

@fstagni fstagni linked a pull request Jul 26, 2023 that will close this issue
@fstagni
Copy link
Contributor

fstagni commented Jul 26, 2023

Prob fixed in #7130

@fstagni fstagni added this to the v8.0 milestone Jul 26, 2023
@fstagni fstagni self-assigned this Jul 26, 2023
@fstagni
Copy link
Contributor

fstagni commented Aug 2, 2023

Should be fixed.

@fstagni fstagni closed this as completed Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants