Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

[Hotfix] Short-term Solution to GW Port Issues #504

Merged
merged 19 commits into from
Dec 4, 2020
Merged

[Hotfix] Short-term Solution to GW Port Issues #504

merged 19 commits into from
Dec 4, 2020

Conversation

kevin-zhonghao
Copy link
Contributor

@kevin-zhonghao kevin-zhonghao commented Dec 4, 2020 •
edited by xieus
Loading

Issue Description: #495 #498

Solution:

  1. Issue 495: Adjusted the logic sequence of code execution and commented the operation of deleting gateway port in port manager
  2. Issue 498: Store subnet entity in db before creating gateway port in PM, and then update subnet in the end

@kevin-zhonghao
commit message
24b9b76
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
08ec421 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
2507b8f 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
8cab6bf 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
d9d2a58 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
daba98e 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
b813735 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
fd9a984 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
e44875a 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
98dcf0d 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
a9c3a47 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
f058bd1 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
08d4a6f 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
ef0b4fe 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
bf29fc7 
…o new_master
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor int…
b23608a 
…o new_master
@kevin-zhonghao
temporary solution for issue 495 and 498
d573fbf
This was linked to issues Dec 4, 2020
[Subnet] Delete a subnet got 404 error from Route Manager #495
Closed
[Route Manager] Add/Remove Neutron gateway port got 500 from Subnet Manager #498
Closed
@xieus xieus added this to the Version 1.0.2020.11.30 milestone Dec 4, 2020
@xieus xieus added the bug Something isn't working label Dec 4, 2020
@xieus xieus requested a review from cj-chung December 4, 2020 00:20
@xieus xieus self-requested a review December 4, 2020 03:53
xieus
xieus approved these changes Dec 4, 2020
View reviewed changes
Copy link
Contributor

@xieus xieus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for moving this fast @kevin-zhonghao This is a very good example of agile development.

@xieus xieus removed a link to an issue Dec 4, 2020
[Subnet] Delete a subnet got 404 error from Route Manager #495
Closed
@xieus xieus changed the title [Issue Solution] Fix issue 495 and issue 498 [Hotfix] Short-term Solution to GW Port Issues Dec 4, 2020
@xieus xieus merged commit 977ac09 into futurewei-cloud:master Dec 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@xieus xieus xieus approved these changes

@cj-chung cj-chung Awaiting requested review from cj-chung

Assignees

@kevin-zhonghao kevin-zhonghao

Labels
bug Something isn't working
Projects
None yet
Milestone
Version 0.10.2020.11.30
Development

Successfully merging this pull request may close these issues.

[Route Manager] Add/Remove Neutron gateway port got 500 from Subnet Manager
2 participants
@kevin-zhonghao @xieus