Skip to content

Fix last ocurrence of unnecessary boxing detected by findbugs#368

Closed
rsafonseca wants to merge 1 commit into
apache:masterfrom
rsafonseca:findbugs7
Closed

Fix last ocurrence of unnecessary boxing detected by findbugs#368
rsafonseca wants to merge 1 commit into
apache:masterfrom
rsafonseca:findbugs7

Conversation

@rsafonseca
Copy link
Copy Markdown
Contributor

CiscoVnmcResource.java:266, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High
Boxing/unboxing to parse a primitive com.cloud.network.resource.CiscoVnmcResource.getIpRangeFromCidr(String)

CiscoVnmcResource.java:266, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High
Boxing/unboxing to parse a primitive com.cloud.network.resource.CiscoVnmcResource.getIpRangeFromCidr(String)
@asfbot
Copy link
Copy Markdown

asfbot commented Jun 7, 2015

cloudstack-pull-requests #321 FAILURE
Looks like there's a problem with this pull request

@DaanHoogland
Copy link
Copy Markdown
Contributor

lgtm

@asfgit asfgit closed this in 9bac84a Jun 8, 2015
yadvr added a commit that referenced this pull request Jan 20, 2021
This uses keys based heuristics to migrate all translations from legacy UI to Primate.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Apr 11, 2024
* Update VM's state if powerstate & state are not in sync

* Add unit tests

* some code improvements for instacne / power state check

* Update power state after vm stop confirmation (as power state 'PowerOn' is kept after vm stop and not updated later)

* Reset the power state update counter before migrate, to allow power state sync to proper state / host

* Do not consider transitional states (Starting, Stopping) to check power state sync

* set powerstate to off for all vm types

---------

Co-authored-by: Suresh Kumar Anaparti <suresh.anaparti@shapeblue.com>
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.

3 participants