Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix vpcnatgw image is not synced #4264

Merged
merged 1 commit into from
Jul 8, 2024
Merged

fix vpcnatgw image is not synced #4264

merged 1 commit into from
Jul 8, 2024

Conversation

oilbeater
Copy link
Collaborator

@oilbeater oilbeater commented Jul 8, 2024

Pull Request

What type of this PR

The vpc-nat-gw image will only be updated when both the ovn-vpc-nat-gw-config and ovn-vpc-nat-config are changed. This PR split the logical, now changing the vpc-nat-gw image only need to change the ovn-vpc-nat-config.

I have write an e2e about this and it passed. However, when running together with other cases I meet other issues when quickly adding and deleting vpc and subnet with same name. I will fix them later and add the e2e together in another PR.

  • Bug fixes

Which issue(s) this PR fixes

Fixes #(issue-number)

Signed-off-by: oilbeater <liumengxinfly@gmail.com>
@oilbeater oilbeater requested a review from bobz965 July 8, 2024 07:13
@oilbeater oilbeater added bug Something isn't working need backport labels Jul 8, 2024
@oilbeater oilbeater merged commit e25bf15 into master Jul 8, 2024
61 of 62 checks passed
@oilbeater oilbeater deleted the fix/vpc-nat-image branch July 8, 2024 10:15
oilbeater added a commit that referenced this pull request Jul 9, 2024
Signed-off-by: oilbeater <liumengxinfly@gmail.com>
bobz965 pushed a commit that referenced this pull request Jul 22, 2024
Signed-off-by: oilbeater <liumengxinfly@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need backport
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants