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

swift: Remove deprecated option allow_versions #1971

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwu83tw
Copy link
Contributor

@kwu83tw kwu83tw commented Jan 9, 2019

Start to log deprecation warning since Queens[1].
swift-c: Option allow_versions is deprecated. Configure the versioned_writes
middleware in the proxy-server instead. This option will be ignored in a future
release. Please refer the object versioning documentation for more
information[2].

[1] https://docs.openstack.org/releasenotes/swift/queens.html#relnotes-2-16-0-stable-queens
[2] https://docs.openstack.org/swift/latest/overview_object_versioning.html

@kwu83tw
Copy link
Contributor Author

kwu83tw commented Jan 9, 2019

According to the comment from Matt, there might be a risk to deprecate this option if customer enabling 'allow_versions' before Kilo release(https://gerrit.suse.provo.cloud/#/c/5474/). Need further update from him before merging this PR.

@matthewoliver
Copy link
Contributor

Thanks @kwu83tw, let me go work on an upstream patch and/or talk to other upstream cores to cover this.
Obviously this isn't an issue for green field customers, only customers pre-kilo.

I'll report back here, and on the gerrit patch once something is decided and/or landed and backported ;)

Maybe we should add the 'do not merge yet' label.

@matthewoliver
Copy link
Contributor

I've raised a bug upstream to track this issue: https://bugs.launchpad.net/swift/+bug/1811044

Start to log deprecation warning since Queens[1].
swift-c: Option allow_versions is deprecated. Configure the versioned_writes
middleware in the proxy-server instead. This option will be ignored in a future
release. Please refer the object versioning documentation for more
information[2].

[1] https://docs.openstack.org/releasenotes/swift/queens.html#relnotes-2-16-0-stable-queens
[2] https://docs.openstack.org/swift/latest/overview_object_versioning.html
Copy link
Contributor

@cmurphy cmurphy left a comment

Choose a reason for hiding this comment

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

In the ardana equivalent I've advocated for not worrying about this deprecation for now and living with it until the next round of deprecation revisions, when hopefully upstream has created a generalized solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants