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

Update installation docs #154

Merged
merged 4 commits into from
Sep 30, 2024
Merged

Update installation docs #154

merged 4 commits into from
Sep 30, 2024

Conversation

stuartmcalpine
Copy link
Collaborator

Update installation docs to include into about desc-python-bleed

Copy link
Collaborator

@JoanneBogart JoanneBogart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've asked for one addition (unless there is similar text somewhere else; I couldn't find it) with suggested text. Feel free to improve the text or move it somewhere else; I just think something like that should be said somewhere. One problem with putting it here is that the concept of owner type hasn't been introduced yet. Maybe it should be mentioned in the Owner and Owner type section of the first tutorial. Or maybe something should be said in both places. Otherwise it looks good.

sqlalchemy.url : postgresql://<username>@dataregistry-release-test-loadbalancer.mcalpine-test.development.svc.spin.nersc.org:5432/desc_data_registry

where ``<username>`` should either be ``reg_writer`` or ``reg_reader``,
depending on what account you have access to.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to add a note here (or somewhere) something like this:
Note: reg_writer cannot write to production (schema lsst_desc_production). There is another more privileged account for that. If you need to register production entries, please consult one of the dataregistry admins. Both reg_reader and reg_writer can write to the tutorial version of production (schema tutorial_production).

@stuartmcalpine stuartmcalpine merged commit a062735 into main Sep 30, 2024
26 checks passed
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 this pull request may close these issues.

2 participants