-
Notifications
You must be signed in to change notification settings - Fork 149
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: update function configuration #4560
fix: update function configuration #4560
Conversation
Signed-off-by: sivchari <shibuuuu5@gmail.com>
33328a0
to
c8f35f1
Compare
VpcConfig: &types.VpcConfig{ | ||
SecurityGroupIds: fm.Spec.VPCConfig.SecurityGroupIDs, | ||
SubnetIds: fm.Spec.VPCConfig.SubnetIDs, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should check nullity for fm.Spec.VPCConfig
before call to its fields
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed it.
Signed-off-by: sivchari <shibuuuu5@gmail.com>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #4560 +/- ##
==========================================
+ Coverage 29.91% 29.93% +0.01%
==========================================
Files 220 220
Lines 25873 25871 -2
==========================================
+ Hits 7741 7745 +4
+ Misses 17484 17478 -6
Partials 648 648
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch, thank you 👏
* fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com>
* fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com>
* fix: update function configuration (#4560) * fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com> * Release v0.45.2 (#4561) Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com> Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com> Co-authored-by: sivchari <shibuuuu5@gmail.com>
* fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com> Signed-off-by: 徳田 真之介 <tokuda_shinnosuke@cyberagent.co.jp>
* fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com> Signed-off-by: sZma5a <masaaki.haribote@gmail.com>
* fix: update function configuration Signed-off-by: sivchari <shibuuuu5@gmail.com> * update: check if vpc is non-nil Signed-off-by: sivchari <shibuuuu5@gmail.com> --------- Signed-off-by: sivchari <shibuuuu5@gmail.com> Signed-off-by: moko-poi <mokopoi44@gmail.com>
What this PR does / why we need it:
@khanhtc1202
I'm so sorry, update function configuration has been already defined and called. So I deleted my changes.
And I fixed order to call UpdateFunctionCode and UpdateFunctionConfiguration. Because UpdateFunctionConfiguration is failed, if Lambda state is pending.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: