Skip to content

Coverity regressions per 10 aug 2015#681

Merged
asfgit merged 2 commits into
apache:masterfrom
DaanHoogland:coverity-regressions
Aug 13, 2015
Merged

Coverity regressions per 10 aug 2015#681
asfgit merged 2 commits into
apache:masterfrom
DaanHoogland:coverity-regressions

Conversation

@DaanHoogland
Copy link
Copy Markdown
Contributor

Not all are in here, the db upgrade code seems to stay the main pitfall.

@asfbot
Copy link
Copy Markdown

asfbot commented Aug 11, 2015

cloudstack-pull-rats #262 SUCCESS
This pull request looks good

@asfbot
Copy link
Copy Markdown

asfbot commented Aug 11, 2015

cloudstack-pull-requests #959 SUCCESS
This pull request looks good

@asfbot
Copy link
Copy Markdown

asfbot commented Aug 12, 2015

cloudstack-pull-analysis #195 SUCCESS
This pull request looks good

@koushik-das
Copy link
Copy Markdown
Contributor

LGTM

@karuturi
Copy link
Copy Markdown
Member

Thanks for fixing these @DaanHoogland. 👍

@asfgit asfgit merged commit 1ab3b96 into apache:master Aug 13, 2015
asfgit pushed a commit that referenced this pull request Aug 13, 2015
Coverity regressions per 10 aug 2015Not all are in here, the db upgrade code seems to stay the main pitfall.

* pr/681:
  coverity 1315775: proper getting of networkLabel
  coverity 1315774: improvement of code to negate false positive

Signed-off-by: Daan Hoogland <daan@onecht.net>
yadvr pushed a commit that referenced this pull request Jan 20, 2021
Use updateLoadBalancerRule instead editLoadBalancerRule
Fix typo in deleteTags

Co-authored-by: Rakesh Venkatesh <r.venkatesh@global.leaseweb.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@DaanHoogland DaanHoogland deleted the coverity-regressions branch August 16, 2023 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants