-
-
Notifications
You must be signed in to change notification settings - Fork 197
Update zope-manager-users.md with addzopeuser script for docker setups #1869
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 zope-manager-users.md with addzopeuser script for docker setups #1869
Conversation
stevepiercy
commented
Feb 19, 2025
@acsr would you please request a review from someone who has knowledge of this scenario? The technical content is outside my knowledge and experience. I can do English and MyST grammar and syntax, but that's about it. Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"with a non existing user-ID" is a bit confusing here. This addzopeuser command is the same as the one mentioned above, it's just a different procedure for running it inside the Docker container. so we can copy some of the same explanations from above about how it behaves if the user already exists.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a path, not a URL. It's relative to the hostname where the Docker container is served via Traefik.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly I've never noticed this or run into a situation where it was necessary for a new Manager user to have the Owner role on the Zope root. It feels like unnecessary information. It's also nothing unique to using this script within the docker container.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we remove this, given the solution that was merged in plone/cookieplone-templates#154?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a different topic which should be covered in docs about the Docker stack included in the cookieplone templates, not in this chapter about Zope Manager users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what these notes are about
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The conf file can also be zope.conf or zeo.conf depending on the configuration. I submitted plone/plone-backend#175 which will make it so you can just run ./docker-entrypoint.sh addzopeuser userid password -- but that won't be available until it's merged and the next Plone 6.1.x image is released.
@davisagli I moved over all my notes from the closed PR as a first step. I fully agree that there are duplicate informations and some stuff can be moved to the deployment Troubleshooting FAQ. I need to review your comments in detail later and I therefore changed this PR into draft status. I am also happy to see progress in other changes making some of the remarks obsolete.
This makes it easier to run the addzopeuser script with the correct conf. Relevant to plone/documentation#1869
This makes it easier to run the addzopeuser script with the correct conf. Relevant to plone/documentation#1869
...#178) This makes it easier to run the addzopeuser script with the correct conf. Relevant to plone/documentation#1869
...script-for-docker-setups
Uh oh!
There was an error while loading. Please reload this page.
Moved my notes from the original Pull Request #1703 related to ticket #1702
📚 Documentation preview 📚: https://plone6--1869.org.readthedocs.build/