Skip to content

Attempt to resolve ovm3 jenkins failure#497

Closed
rsafonseca wants to merge 6 commits into
apache:masterfrom
rsafonseca:fixjenkinsovm3
Closed

Attempt to resolve ovm3 jenkins failure#497
rsafonseca wants to merge 6 commits into
apache:masterfrom
rsafonseca:fixjenkinsovm3

Conversation

@rsafonseca

@rsafonseca rsafonseca commented Jun 21, 2015

Copy link
Copy Markdown
Contributor

Resolve unsafe cast warnings reported by lint checker
Add some debug config to compiler plugin

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #552 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #557 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #561 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #562 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #578 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@rsafonseca

rsafonseca commented Jun 21, 2015

Copy link
Copy Markdown
Contributor Author

Just as i said travis was stable, got a lot of failures haha :)
Upon looking into it, travis didn't fail, it is correctly reporting that i killed ovm3 lol, so travis fix is working as expected.. this PR is the one who isn't
Will fix soon, i can now reproduce the issue locally :)

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #605 ABORTED

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@rsafonseca

rsafonseca commented Jun 21, 2015

Copy link
Copy Markdown
Contributor Author

Victory!
One of the problem nodes just compiled it successfully and ran until the end without problems
Will now get started on cleaning up the PR and just leave the fix and a couple other minor improvements made along the way

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@rsafonseca
Specify all external dependencies in pom
2f274e7 
Upgrade to lang3 and change usage of deprecated xmlparse method

Add another missing dependency
@rsafonseca
Root cause of failure is memory abuse in XenTest.java class ;) This s…
c765e25 
…hould fix it
@rsafonseca

rsafonseca commented Jun 21, 2015

Copy link
Copy Markdown
Contributor Author

@bhaisaab you can go ahead and merge now :)

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Jun 21, 2015

Copy link
Copy Markdown

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

karuturi
karuturi reviewed Jun 22, 2015
View reviewed changes
Comment thread plugins/hypervisors/ovm3/pom.xml Outdated

@karuturi karuturi Jun 22, 2015

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we resuse the version number(cs.xmlrpc.version) from parent pom?

@rsafonseca rsafonseca Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We can and should, i just kept this one the way it was, version was already specified in the version currently in master, will update

@rsafonseca

rsafonseca commented Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

@karuturi done in a nicer way, as requested :)
FYI, the only thing that is needed to actually fix the ovm3 problem is the changes to plugins/hypervisors/ovm3/src/test/java/com/cloud/hypervisor/ovm3/objects/XenTest.java

All the rest are just minor improvements made along the testing of the problem, but none are related to actually fixing it. The minor improvements i referred to are simply getting rid of some lint checker warnings about serializable classes not having the serialversionuid and doing redundant explicit casts. Also added a couple of non declared dependencies to the pom, and that's it :)

@asfbot

asfbot commented Jun 22, 2015

Copy link
Copy Markdown

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

@asfgit asfgit closed this in fe4f0b1 Jun 22, 2015
@karuturi

karuturi commented Jun 23, 2015

Copy link
Copy Markdown
Member

Thanks for the update @rsafonseca

kishankavala pushed a commit to kishankavala/cloudstack that referenced this pull request Jun 25, 2015
@rsafonseca @kishankavala
Specify all dependency versions in pom.xml
05e2c2b 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes apache#497
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Jan 27, 2025
@vishesh92
FR82 Phase 1 & 2: Improve logging to include more identifiable inform…
0473348 
…ation (apache#497)

* Improve logging to include more identifiable information for kvm plugin

* Update logging for scaleio plugin

* Improve logging to include more identifiable information for default volume storage plugin

* Improve logging to include more identifiable information for agent managers

* Improve logging to include more identifiable information for Listeners

* Replace ids with objects or uuids

* Improve logging to include more identifiable information for engine

* Improve logging to include more identifiable information for server

* Improve logging to include more identifiable information for plugins

* Improve logging to include more identifiable information for Cmd classes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rsafonseca @asfbot @yadvr @karuturi