[ovn-octavia-provider] Job pep8 failing due to bandit new lint rule

[Fernando Royo]

Pep8 jobs are failing in master and stables branches after update bandit to 1.7.5. Basically it is doing a new lint rule checking timeout is specified in any request.

The rule B113 is marked as a warning but it making the job failing.

[1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877357
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/1878eb4c21040ee34cde8a1aae6d1fb8993ae332
Submitter: "Zuul (22348)"
Branch: master

commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877462

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877463

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877464

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877465

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877466

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877462
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/68251e4b413fa8295a2de43d2f52bc8b96c92580
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 68251e4b413fa8295a2de43d2f52bc8b96c92580
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8
    (cherry picked from commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider 4.0.0.0rc2 release candidate.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877465
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/17532ec46730007aa1d099114b08f86df1db4d1a
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 17532ec46730007aa1d099114b08f86df1db4d1a
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8
    (cherry picked from commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877466
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/8d9be2c3a006abfdde08e85f94c2042f0235c6d8
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 8d9be2c3a006abfdde08e85f94c2042f0235c6d8
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8
    (cherry picked from commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877463
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/75b84560c486cae96789b154e41123aed9dd5c6a
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 75b84560c486cae96789b154e41123aed9dd5c6a
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8
    (cherry picked from commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/877464
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/730843bd66a7ebf8b67ae18f7bcc1434d33f4e13
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 730843bd66a7ebf8b67ae18f7bcc1434d33f4e13
Author: Fernando Royo <email address hidden>
Date: Tue Mar 14 12:18:34 2023 +0100

    Fix broken pep8 jobs due to bandit 1.7.5 updated version

    With the latest version of bandit (1.7.5), a new lint rule has been
    introduced that checks the inclusion of the timeout parameter for
    every "requests" call [1].

    So B113 lint rule[2] needs to be skipped or code adapted, this patch
    add the timeout parameter to the put/get requests.

    [1] https://github.com/PyCQA/bandit/commit/5ff73ff8ff956df7d63fde49c3bd671db8e821eb
    [2] https://bandit.readthedocs.io/en/latest/plugins/b113_request_without_timeout.html

    Closes-bug: #2011573
    Change-Id: I341faedbf7e237eed176e0d3ed3586b8d2c2cbb8
    (cherry picked from commit 1878eb4c21040ee34cde8a1aae6d1fb8993ae332)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider 1.3.1 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider 3.1.1 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider 2.1.1 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider 5.0.0.0rc1 release candidate.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ovn-octavia-provider wallaby-eom release.