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

JBTM-1365. REST-AT integration #303

Merged
merged 1 commit into from May 26, 2013

Conversation

Projects
None yet
4 participants
@gytis
Copy link
Member

commented May 25, 2013

No description provided.

@jbosstm-bot

This comment has been minimized.

Copy link

commented May 25, 2013

Started testing this pull request: http://172.17.131.2/job/btny-pulls-narayana/530/

@jbosstm-bot

This comment has been minimized.

Copy link

commented May 25, 2013

Tests failed: RTS AS Integration Test failed

@gytis

This comment has been minimized.

Copy link
Member Author

commented May 25, 2013

Failed to boot up RTS subsystem, because undertow server was not loaded yet.

@gytis

This comment has been minimized.

Copy link
Member Author

commented May 25, 2013

I think adding .addDependency(UndertowService.virtualHostName("default-server", "default-host")) in RTSSubsystemAdd.performBoottime should solve the problem. However, I wouldn't be able to print explanatory message what's happening and put the link to the JIRA...

@paulrobinson

This comment has been minimized.

Copy link
Contributor

commented May 25, 2013

I think we just have to go without the nice message. Hopefully this will be resolved before people start doing anything more advanced with it.

Your dependency plan looks right.

@gytis

This comment has been minimized.

Copy link
Member Author

commented May 25, 2013

The problem is, that I cannot put the message if dependency does not exist. AS does than checking...

@paulrobinson

This comment has been minimized.

Copy link
Contributor

commented May 25, 2013

Yeah sure, I think we will have to go ahead without the message.

On 25 May 2013, at 17:46, Gytis Trikleris notifications@github.com wrote:

The problem is, that I cannot put the message if dependency does not exist. AS does than checking...


Reply to this email directly or view it on GitHub.

@jbosstm-bot

This comment has been minimized.

Copy link

commented May 26, 2013

Started testing this pull request: http://172.17.131.2/job/btny-pulls-narayana/531/

paulrobinson added a commit that referenced this pull request May 26, 2013

Merge pull request #303 from Gytis/master-JBTM-1365
JBTM-1365. REST-AT integration

@paulrobinson paulrobinson merged commit 7f79d49 into jbosstm:master May 26, 2013

@tomjenkinson

This comment has been minimized.

Copy link
Contributor

commented on scripts/hudson/narayana.sh in 3493005 Jul 3, 2013

Hi gytis,

The function you defined is called rest_at_integration_tests not rest_at_integration_test

Hope that helps,
Tom

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.