neutron-dynamic-routing broken by introduction of agent_timestamp to _log_heartbeat()

Bug #1833589 reported by Ryan Tidwell
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Brian Haley

Bug Description

neutron-dynamic-routing is broken due to merging https://review.opendev.org/#/c/665426/, which adds the argument 'agent_timestamp' to _log_heartbeat(). All neutron-dynamic-routing jobs are failing with errors like these http://paste.openstack.org/show/753229/.

Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Actually this method is used in several projects [1]: networking-old, neutron-dynamic-routing

[1] http://codesearch.openstack.org/?q=create_or_update_agent&i=nope&files=&repos=

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

Fix proposed to branch: master
Review: https://review.opendev.org/666654

Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
status: New → In Progress
Revision history for this message
Brian Haley (brian-haley) wrote :

https://review.opendev.org/#/c/666224/ was already approved, there was no bug filed since the code had just been submitted.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Rodolfo Alonso Hernandez (<email address hidden>) on branch: master
Review: https://review.opendev.org/666654
Reason: Superseded by https://review.opendev.org/#/c/666224

Changed in neutron:
assignee: Rodolfo Alonso (rodolfo-alonso-hernandez) → Adit Sarfaty (asarfaty)
importance: Undecided → High
Changed in neutron:
assignee: Adit Sarfaty (asarfaty) → Brian Haley (brian-haley)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/666224
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=0963ce94e01028f8aec3b3af607a2c4d322ec62a
Submitter: Zuul
Branch: master

commit 0963ce94e01028f8aec3b3af607a2c4d322ec62a
Author: Adit Sarfaty <email address hidden>
Date: Wed Jun 19 08:34:16 2019 +0300

    Add default value to agent create_or_update_agent new param

    Commit Ifc88dfb3041aa07b197f395172b69399796ba46a recently added a
    new agent_timestamp parameter to create_or_update_agent.
    As some other projects use this api, the change should have been
    backwards compatible.
    Adding default to None will solve this issue.

    Closes-Bug: #1833589
    Change-Id: I2d17f8eaf4d74ae2081f82e3c1a09bdc80b1d701

Changed in neutron:
status: In Progress → Fix Released
tags: added: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 15.0.0.0b1

This issue was fixed in the openstack/neutron 15.0.0.0b1 development milestone.

tags: removed: neutron-proactive-backport-potential
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.