Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-221846787
### ACS CI BVT Run
**Sumarry:**
Build Number 56
Hypervisor xenserver
NetworkType Advanced
Passed=72
Failed=1
Skipped=3
_
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1376
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218959204
LGTM
tag:mergeready
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218958767
@swill Done. Jenkins and Travis are green now. Ready to merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218808022
@milamberspace can you do a force push again. I have pushed fixes to
Jenkins and Travis this morning, so with a new push we should be able to get
this green. Thx...
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218376280
@swill @milamberspace weird, we've lost our Travis integration with PRs
here. https://travis-ci.org/apache/cloudstack/ says "The repository at
apache/cloudstack was n
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218375832
@swill conflicts are fixed. Ready to merge. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-218367016
#1515 has now been merged (sorry for the delay). Once you have a chance to
fix the merge conflicts, I can get this merged. Thx...
---
If your project is set up for
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-216889967
This PR currently has merge conflicts, but #1515 is next in line, so you
may want to wait till it is merged before you fix these conflicts.
---
If your project is
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-216221653
LGTM
tag:easypr
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user swill commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1376#discussion_r60733737
--- Diff: tools/transifex/.tx/config ---
@@ -89,3 +89,41 @@ trans.pt_BR = work-dir/messages_pt_BR.properties
trans.ru_RU = work-dir/messages_ru_RU.prop
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-213336322
@swill Re-push-force is done.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user milamberspace commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1376#discussion_r60708800
--- Diff: tools/transifex/.tx/config ---
@@ -89,3 +89,41 @@ trans.pt_BR = work-dir/messages_pt_BR.properties
trans.ru_RU = work-dir/messages_ru
Github user swill commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1376#discussion_r60653156
--- Diff: tools/transifex/.tx/config ---
@@ -89,3 +89,41 @@ trans.pt_BR = work-dir/messages_pt_BR.properties
trans.ru_RU = work-dir/messages_ru_RU.prop
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-213100441
That is fine. @milamberspace would you mind just doing a force push again
to kick off jenkins. I have seen other jenkins runs passing so I think jenkins
is just not
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-213099792
@swill, @milamberspace sorry, the build is gone, I didn't react quick enough
---
If your project is set up for it, you can reply to this email and have your
re
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-212585886
@DaanHoogland do you know why jenkins is failing here?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-212572821
@swill squash & force push done.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your proje
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-212530190
Jenkins failed with a timeout (not related to your code). Can you squash
your commits and do a force push again? Sorry for the runaround..
---
If your project is s
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-212324488
@swill I've just rebase from master, update the L10N files and push --force
this PR.
cc @koushik-das
---
If your project is set up for it, you can reply
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-211917846
Thanks koushik. I tend to agree with you on this one. I will add it to my
merge list. Thanks...
---
If your project is set up for it, you can reply to this email
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-211859221
The changes are only limited to UI, so there is no need to run
smoke/integration tests.
Cursory look at the changes looks fine.
@swill I think these shou
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1376#issuecomment-197977522
### ACS CI BVT Run
**Sumarry:**
Build Number 107
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=15
Skipped=4
GitHub user milamberspace opened a pull request:
https://github.com/apache/cloudstack/pull/1376
L10n update master 20160127
cc @remibergsma @bhaisaab @DaanHoogland
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/milamberspace/clo
24 matches
Mail list logo
