-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add n-vol #1
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
Add n-vol #1
Conversation
|
Thank you for contributing to OpenStack! openstack-dev/devstack uses Gerrit for code review. Please visit http://wiki.openstack.org/GerritWorkflow and follow the instructions there to upload your change to Gerrit. |
1 similar comment
|
Thank you for contributing to OpenStack! openstack-dev/devstack uses Gerrit for code review. Please visit http://wiki.openstack.org/GerritWorkflow and follow the instructions there to upload your change to Gerrit. |
Service fixes
Updated README
This reverts commit ef5ebed. The problem here is a backwards-incompatible change to configure_auth_token_middleware. Plugins are still passing a "signing_dir" which is interpreted now as the "section" argument ... this leads to an interesting red-herring issue; because "v" is a gnu sed command for checking the version, a signing_dir of "/var/..." (as done in most plugins) gives the weird error: sed: -e expression #1, char 32: expected newer version of sed I think we'll either need a new function, or dummy arguments to get this back in. Change-Id: I2098d4eb2747282622cf486fa7dbf216f932f58b
I941ef5ea90970a0901236afe81c551aaf24ac1d8 added a sed command that should match and delete path values but used '/' as sed separator. This leads to error in unstack.sh runs when the path also contains '/': +./unstack.sh:main:188 sudo sed -i '/directory=/opt/stack/ d' /etc/gitconfig sed: -e expression #1, char 13: unknown command: `o' So this patch replace '/' separator with '+'. Change-Id: I06811c0d9ee7ecddf84ef1c6dd6cff5129dbf4b1
I941ef5ea90970a0901236afe81c551aaf24ac1d8 added a sed command that should match and delete path values but used '/' as sed separator. This leads to error in unstack.sh runs when the path also contains '/': +./unstack.sh:main:188 sudo sed -i '/directory=/opt/stack/ d' /etc/gitconfig sed: -e expression #1, char 13: unknown command: `o' So this patch replace '/' separator with '+'. Change-Id: I06811c0d9ee7ecddf84ef1c6dd6cff5129dbf4b1 (cherry picked from commit 7191c5e)
I941ef5ea90970a0901236afe81c551aaf24ac1d8 added a sed command that should match and delete path values but used '/' as sed separator. This leads to error in unstack.sh runs when the path also contains '/': +./unstack.sh:main:188 sudo sed -i '/directory=/opt/stack/ d' /etc/gitconfig sed: -e expression #1, char 13: unknown command: `o' So this patch replace '/' separator with '+'. Change-Id: I06811c0d9ee7ecddf84ef1c6dd6cff5129dbf4b1 (cherry picked from commit 7191c5e) (cherry picked from commit 7cfe946)
I941ef5ea90970a0901236afe81c551aaf24ac1d8 added a sed command that should match and delete path values but used '/' as sed separator. This leads to error in unstack.sh runs when the path also contains '/': +./unstack.sh:main:188 sudo sed -i '/directory=/opt/stack/ d' /etc/gitconfig sed: -e expression openstack#1, char 13: unknown command: `o' So this patch replace '/' separator with '+'. Change-Id: I06811c0d9ee7ecddf84ef1c6dd6cff5129dbf4b1
n-vol need glance if you only install n-vol