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

Revert ceph-csi v2.0 update #171

Merged
merged 1 commit into from Mar 24, 2020
Merged

Revert ceph-csi v2.0 update #171

merged 1 commit into from Mar 24, 2020

Conversation

johnsca
Copy link
Contributor

@johnsca johnsca commented Mar 19, 2020

This reverts PRs #168 and #169.

This can be re-reverted (see LP #1867940) once the Ussuri release of the OpenStack charms are available (targeted for Focal, presumably to be released in May 2020).

This addresses LP #1868150

This reverts PRs #168 and #169.

This can be re-reverted once the Ussuri release of the OpenStack charms
are available (see LP #1867940 [1]).

This addresses LP #1868150 [2]

[1]: https://bugs.launchpad.net/cdk-addons/+bug/1867940
[2]: https://bugs.launchpad.net/cdk-addons/+bug/1868150
@johnsca
Copy link
Contributor Author

johnsca commented Mar 19, 2020

Tested on AWS with the 1.18/edge charms. Still has the permission issue with RWX mounts in non-root pods, as expected (see LP #1866262 but otherwise functions as it should.

Copy link
Member

@kwmonroe kwmonroe left a comment

Choose a reason for hiding this comment

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

LGTM

@kwmonroe kwmonroe merged commit bbf4a31 into master Mar 24, 2020
@kwmonroe kwmonroe deleted the johnsca/revert-168 branch March 24, 2020 15:12
johnsca added a commit that referenced this pull request Apr 9, 2020
This got incorrectly reverted in #171. Also pin to the release tag
rather than the branch, since point releases might re-use the branch.

Fixes https://bugs.launchpad.net/cdk-addons/+bug/1871945
kwmonroe pushed a commit that referenced this pull request Apr 9, 2020
This got incorrectly reverted in #171. Also pin to the release tag
rather than the branch, since point releases might re-use the branch.

Fixes https://bugs.launchpad.net/cdk-addons/+bug/1871945
johnsca added a commit that referenced this pull request Apr 13, 2020
This got incorrectly reverted in #171. Also pin to the release tag
rather than the branch, since point releases might re-use the branch.

Fixes https://bugs.launchpad.net/cdk-addons/+bug/1871945
kwmonroe added a commit that referenced this pull request Apr 13, 2020
…rovider (#176)

* Re-apply new provisioner value for OpenStack SC (#174)

This got incorrectly reverted in #171. Also pin to the release tag
rather than the branch, since point releases might re-use the branch.

Fixes https://bugs.launchpad.net/cdk-addons/+bug/1871945

* track openstack release branch instead of tag (#175)

Co-authored-by: Kevin W Monroe <kevin.monroe@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants