Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78455032
@bhaisaab @likitha cool thanks!
---
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 bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78449863
@snuf @likitha yeah, RAT expects a certain format. Just fixed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user runseb commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78339080
That's ok @bhaisaab you will be fine, @snuf is the huggy type you two can
get emotional about that code at the next cloudstack day.
---
If your project is set up for
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78436606
@likitha Hmm the format is indeed different, though the content is the
same, mine is the one that comes with eclipse and has a full line of * at the
top and bottom and i
Github user likitha commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78435972
Seems to have caused the RAT check to fail. I do see the license headers in
the new files, so I guess the format is not an exact match?
(https://builds.apache.org/job
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/113
---
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 bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78428527
@runseb true that :)
@snuf we're good as long as the bugs are in the plugin itself and don't
affect the core. Looking at the code, does not look like the sev
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78336094
@bhaisaab thanks man!, it's bound to have some bugs and nasty bits here and
there. Already have NSX/NVP integration on the map for the next iteration
though :)
I'll
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78328589
@snuf after a quick look at the diff I feel so emotional with the code I'm
seeing, neat stuff :) :astonished:
---
If your project is set up for it, you can reply t
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/113#issuecomment-78327336
Hey @snuf thanks for that! It makes it easier to everyone to review and
merge your stuff. First thing tomorrow morning!
---
If your project is set up for it, you ca
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/99#issuecomment-78311833
@bhaisaab move this into https://github.com/apache/cloudstack/pull/113
which is a new request, and a bit more condensed.
---
If your project is set up for it, you ca
Github user snuf closed the pull request at:
https://github.com/apache/cloudstack/pull/99
---
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 ena
GitHub user snuf opened a pull request:
https://github.com/apache/cloudstack/pull/113
Ovmsupport 3.2.1/3.3.x rebased and condensed in one single commit.
@bhaisaab
Rebasing the other stuff was a pain (4400 commits behind is no pleasure).
Decided to do a "re-implement" agains
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/99#issuecomment-77811087
@snuf too many merge conflicts if I merge from your branch to master
branch. This PR is for merging your branch to the ovmsupport branch. Please go
ahead and merge it
Github user ghxandsky commented on the pull request:
https://github.com/apache/cloudstack/pull/99#issuecomment-77675935
So good.
---
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
e
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-77004275
Kay happy to report it was a problem in my rollout script which I changed
recently *cough* so everything is fine, itâs just that the change left out
the vlan range, so t
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-76862484
Hey Rohit,
Well all was fine until I did my last two merges from master after doing
the 3.3.x integration of OVM, allocate vnet breaks and rolls back on the DB all
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-76694838
@snuf Funs how about we merge your stuff in?
---
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 pr
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-66262896
Hi Funs,
Thanks for replying. Alright let us know when you've good set of
integration tests, I'm halfway with a new integration testing system
semi-automated
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-66134697
Hi Rohit,
Well there are a couple of things here:
1st: I work on a fork and pull in upstream master and merge that into
ovmsupport (https://github.com/snu
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-65896135
That's great Funs. I see it's already passing smoke tests on TravisCI so we
can merge it on master whenever you advise? If it's functionally working with
good enough c
Github user snuf commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-65893442
Hey Rohit,
Itâs been rebased with master two days ago, so should be a âbreezeâ,
is there a specific point you want me to merge with ?
Cheers,
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/9#issuecomment-65824504
Hey, @snuf can you rebase this against master, let's merge it on master if
it works functionally and the build passes with smoke tests?
---
If your project is set up
23 matches
Mail list logo
