-
Notifications
You must be signed in to change notification settings - Fork 61
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
Review and refactor openstack-base bundle README #144
Review and refactor openstack-base bundle README #144
Conversation
…ack-bundles into review-openstack-base-readme
…ack-bundles into review-openstack-base-readme
stable/openstack-base/README.md
Outdated
|
||
## Ensuring it's working | ||
juju deploy bundle.yaml --overlay openstack-base-spaces-overlay.yaml |
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.
note that once we move to 2.7, you will have to type:
juju deploy ./bundle.yaml - not being explicit about the path will result in an error (unles this is changed pre-stable release)
…enstack-bundles into review-openstack-base-readme
stable/openstack-base/README.md
Outdated
|
||
openstack security group list | ||
http://<openstack-dashboard-ip>/horizon |
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 renders incorrectly when viewing in GH:
http:///horizon
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.
Thank you Chris. 🍪 Fixes committed.
The broken networking scripts can be assisted by #142.
Included is a drive-by improvement of the repository README.
Closed-Bug: #1671929
https://bugs.launchpad.net/charm-guide/+bug/1671929