DeprecationWarning: Flags not at the start of the expression

Bug #1987539 reported by Takashi Natsume
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Takashi Natsume

Bug Description

Running "tox -e py38", the following warning appear.

2022-08-23 20:59:46.250673 | ubuntu-focal | {2} cinder.tests.unit.targets.test_iet_driver.TestIetAdmDriver.test_get_target [0.055716s] ... ok
2022-08-23 20:59:46.250744 | ubuntu-focal |
2022-08-23 20:59:46.250775 | ubuntu-focal | Captured stderr:
2022-08-23 20:59:46.250822 | ubuntu-focal | ~~~~~~~~~~~~~~~~
2022-08-23 20:59:46.250853 | ubuntu-focal | /home/zuul/src/opendev.org/openstack/cinder/cinder/volume/targets/iet.py:63: DeprecationWarning: Flags not at the start of the expression '^tid:(?m)'
2022-08-23 20:59:46.250882 | ubuntu-focal | session_list = re.split('^tid:(?m)', sessions)[1:]
2022-08-23 20:59:46.250909 | ubuntu-focal |

2022-08-23 21:00:07.104432 | ubuntu-focal | {7} cinder.tests.unit.targets.test_iet_driver.TestIetAdmDriver.test_find_sid_cid_for_target [0.053381s] ... ok
2022-08-23 21:00:07.104485 | ubuntu-focal |
2022-08-23 21:00:07.104505 | ubuntu-focal | Captured stderr:
2022-08-23 21:00:07.104523 | ubuntu-focal | ~~~~~~~~~~~~~~~~
2022-08-23 21:00:07.104542 | ubuntu-focal | /home/zuul/src/opendev.org/openstack/cinder/cinder/volume/targets/iet.py:216: DeprecationWarning: Flags not at the start of the expression '^tid:(?m)'
2022-08-23 21:00:07.104575 | ubuntu-focal | session_list = re.split('^tid:(?m)', sessions)[1:]
2022-08-23 21:00:07.104600 | ubuntu-focal |

https://6652f590ce298f0dccf9-a12e67e84744b622e6a13b507a2faa27.ssl.cf5.rackcdn.com/852211/5/check/openstack-tox-py38/b77ca0b/job-output.txt

Tags: tox
summary: - DeprecationWarning: Flags not at the start of the expression '^tid:(?m)'
- but at position 5
+ DeprecationWarning: Flags not at the start of the expression
Changed in cinder:
assignee: nobody → Takashi Natsume (natsume-takashi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/854428

Changed in cinder:
status: New → In Progress
Changed in cinder:
importance: Undecided → Low
tags: added: tox
Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Fix looks good and may merge soon but please remember that
'The IET iSCSI target is deprecated and will be removed in the '
'"V" release. It is recommended to use the LIO or TGT targets '
'instead.'
Cheers

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/854428
Committed: https://opendev.org/openstack/cinder/commit/f6a1b9c3eca1fc20f3f0f2c1cb1db8b2a343564e
Submitter: "Zuul (22348)"
Branch: master

commit f6a1b9c3eca1fc20f3f0f2c1cb1db8b2a343564e
Author: Takashi Natsume <email address hidden>
Date: Thu Aug 25 00:14:12 2022 +0900

    Fix a deprecation warning about regex

    Fix the following deprecation warning.

    * DeprecationWarning: Flags not at the start of
      the expression '^tid:(?m)'

    Change-Id: I04aede6aec12cf89addeb1c9d500f0c623bba1a5
    Closes-Bug: 1987539
    Signed-off-by: Takashi Natsume <email address hidden>

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 21.0.0.0rc1

This issue was fixed in the openstack/cinder 21.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.