Hi Team,
Last week we released glance_store 2.3.0 which adds support for configuring cinder multiple stores as glance backend. While adding functional tests in glance for the same [1], we have noticed that it is failing with some hard requirements from oslo side to use project_id instead of tenant and user_id instead of user. It is really strange behavior as this failure occurs only in functional tests but works properly in the actual environment without any issue. The fix is proposed in glance_store [2] to resolve this issue.
I would like to apply for FFE with this glance_store patch [2] to be approved and release a new version of glance_store 2.3.1.
Kindly provide approval for the same.
[1] https://review.opendev.org/#/c/750144/ [2] https://review.opendev.org/#/c/750131/
Thanks and Regards, Rajat Dhasmana
Hi Team,
The reason for failure is we are suppressing Deprecation warning into error in glance [1] and we are using those deprecated parameters in glance_store. This is the reason why it is only failing in functional tests [2] and not in actual scenarios.
[1] https://opendev.org/openstack/glance/src/branch/master/glance/tests/unit/fix... [2]https://review.opendev.org/#/c/750144/
Thanks & Best Regards,
Abhishek Kekane
On Tue, Sep 8, 2020 at 8:48 PM Rajat Dhasmana rdhasman@redhat.com wrote:
Hi Team,
Last week we released glance_store 2.3.0 which adds support for configuring cinder multiple stores as glance backend. While adding functional tests in glance for the same [1], we have noticed that it is failing with some hard requirements from oslo side to use project_id instead of tenant and user_id instead of user. It is really strange behavior as this failure occurs only in functional tests but works properly in the actual environment without any issue. The fix is proposed in glance_store [2] to resolve this issue.
I would like to apply for FFE with this glance_store patch [2] to be approved and release a new version of glance_store 2.3.1.
Kindly provide approval for the same.
[1] https://review.opendev.org/#/c/750144/ [2] https://review.opendev.org/#/c/750131/
Thanks and Regards, Rajat Dhasmana
Hi Team,
FFE is not needed as the failures were related to tests only and does not impact actual functionality.
Thanks & Best Regards,
Abhishek Kekane
On Tue, Sep 8, 2020 at 8:53 PM Abhishek Kekane akekane@redhat.com wrote:
Hi Team,
The reason for failure is we are suppressing Deprecation warning into error in glance [1] and we are using those deprecated parameters in glance_store. This is the reason why it is only failing in functional tests [2] and not in actual scenarios.
[1] https://opendev.org/openstack/glance/src/branch/master/glance/tests/unit/fix... [2]https://review.opendev.org/#/c/750144/
Thanks & Best Regards,
Abhishek Kekane
On Tue, Sep 8, 2020 at 8:48 PM Rajat Dhasmana rdhasman@redhat.com wrote:
Hi Team,
Last week we released glance_store 2.3.0 which adds support for configuring cinder multiple stores as glance backend. While adding functional tests in glance for the same [1], we have noticed that it is failing with some hard requirements from oslo side to use project_id instead of tenant and user_id instead of user. It is really strange behavior as this failure occurs only in functional tests but works properly in the actual environment without any issue. The fix is proposed in glance_store [2] to resolve this issue.
I would like to apply for FFE with this glance_store patch [2] to be approved and release a new version of glance_store 2.3.1.
Kindly provide approval for the same.
[1] https://review.opendev.org/#/c/750144/ [2] https://review.opendev.org/#/c/750131/
Thanks and Regards, Rajat Dhasmana
participants (2)
-
Abhishek Kekane
-
Rajat Dhasmana