yahoo-eng-team team mailing list archive

[OpenStack Infra <2011573@xxxxxxxxxxxxxxxxxx>]

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 <froyo@xxxxxxxxxx>
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


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2011573

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

Status in neutron:
  Fix Released

Bug description:
  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

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2011573/+subscriptions