Skip to content

Setting up Travis CI #22

Merged
burmanm merged 3 commits intohawkular:masterfrom
rubenvp8510:travis
Jan 2, 2017
Merged

Setting up Travis CI #22
burmanm merged 3 commits intohawkular:masterfrom
rubenvp8510:travis

Conversation

@rubenvp8510
Copy link
Copy Markdown
Contributor

No description provided.

@yaacov
Copy link
Copy Markdown
Contributor

yaacov commented Dec 9, 2016

👍 For fixing the string metrics test

Can you set to test with more then one hawkular version ?
We have a PR to support legacy API #20 and It will be nice if legacy versions also get tested.

@burmanm
Copy link
Copy Markdown
Contributor

burmanm commented Dec 19, 2016

Can you rebase?

@yaacov
Copy link
Copy Markdown
Contributor

yaacov commented Dec 22, 2016

Thanks 👍

@rubenvp8510 rubenvp8510 force-pushed the travis branch 7 times, most recently from 9d34276 to a9edbb0 Compare December 27, 2016 16:28
Fix some metrics test in order to pass Travis.

Fix alerts for work with python 3.4.x

Support two versions of hawkular metrics for testing
Support two versions of hawkular metrics.
@rubenvp8510
Copy link
Copy Markdown
Contributor Author

@burmanm This is ready for another review could you please review it when you get a chance? Thank you!, @yaacov I added tests for version 0.15, I'm not sure if that is enough.

@yaacov
Copy link
Copy Markdown
Contributor

yaacov commented Dec 28, 2016

@rubenvp8510 it's just what I wanted ( ver < 0.16 )

p.s.
I wanted this because the QE team is now using very old 0.8, I hope they will start using 0.21 soon, with the alerts, and I will not need this :-)

@burmanm burmanm merged commit 75feb15 into hawkular:master Jan 2, 2017
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