Github user remibergsma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1071#discussion_r44907228
--- Diff: build/replace.properties ---
@@ -26,4 +26,4 @@ AGENTLOG=logs/agent.log
MSMNTDIR=/mnt
COMPONENTS-SPEC=components.xml
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1061#issuecomment-156984574
@DaanHoogland Yes, please make this PR towards 4.6 please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1062#issuecomment-156984813
@dsclose Can you please make this PR against 4.6 branch? After merge it
will be fwd-merged to master. Thanks!
---
If your project is set up for it, you can
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1059#issuecomment-156985363
@ustcweizhou Can you please make this PR against 4.6 branch? After merge it
will be fwd-merged to master. Thanks!
---
If your project is set up for it, you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1038#issuecomment-156986694
I've run this for some time and it is really cool, thanks @bhaisaab LGTM
If you consider back porting this to 4.5.3, then I think we should
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157168301
LGTM:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component/test_vpc_redundant.py
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1071#issuecomment-157169270
@DaanHoogland @wilderrodrigues @miguelaferreira Can either of you review so
we can merge this and open master for new features?
---
If your project is set up
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/801#issuecomment-157311229
@KrisSterckx @nlivens Guys, we are ready to merge this into master/4.7 now.
Since it was rebased last some weeks ago, I'll do a quick test run against
cu
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/768#issuecomment-157311669
@bhaisaab @abhinandanprateek it would be great to include this in 4.7. Can
you guys rebase it and resolve the conflicts?
@jburwell Can you update your
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/935#issuecomment-157311867
Would be great to include this in 4.7, please ping me when you're ready!
---
If your project is set up for it, you can reply to this email and have your
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/872#issuecomment-157313386
@jayapalu @karuturi Let's get this into 4.7! Can you respond to the
comments please @jayapalu ?
---
If your project is set up for it, you can reply to
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1073#issuecomment-157409023
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first):
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1076#issuecomment-157409637
Hi @nitin-maharana thanks for the fix! Please make this pull request
against 4.6, so it can be fixed there as well. Once merged, it will be forward
merged to
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1062#issuecomment-157410651
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first):
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1060#issuecomment-157411442
@DaanHoogland FYI test results:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1079#issuecomment-157421231
@wilderrodrigues I want to merge this to 4.6, so the one against master
should be closed. As we merge forward now, we need bug fixes against 4.6.
Otherwise it
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1038#issuecomment-157421614
@DaanHoogland Because @bhaisaab wants it in 4.5.3. Otherwise people will
upgrade from 4.5.3 to say 4.6.2 and lose a feature.
---
If your project is set up for
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1076#issuecomment-157498727
@nitin-maharana You can close this one. When the 4.6 one is merged, it will
be forwarded merged to master automatically.
---
If your project is set up for it
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1079#issuecomment-157670606
Gave LGTM on #1062, extending it here.
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/801#issuecomment-157680581
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first now that master moved on):
```
nosetests --with-marvin
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/801#issuecomment-157681280
Congrats @nlivens @KrisSterckx this is now merged in master and will be
included in 4.7.0. Thanks for all the work to get this in :-)
---
If your project is
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/935#issuecomment-157700801
@nvazquez Can you look at this build failure:
http://jenkins.buildacloud.org/job/build-master-noredist/4746/console
Does the slave need the 5.5 jar or
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/935#issuecomment-157701917
According to Jenkins the build was fine:
```
Maven: -B -f
/home/jenkins/jenkins-slave/workspace/cloudstack-pull-analysis/pom.xml
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1038#issuecomment-158021203
@bhaisaab I'm OK with merging this to master now, but that will also mean
it will not end up in 4.6. I think @DaanHoogland already said that no new
fea
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1072#issuecomment-158053349
LGTM, did an upgrade from `4.4.4` to `4.6.1-SNAPSHOT` and that worked now
with only the `4.6` template registered. This was done with XenServer
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1070#issuecomment-158116184
LGTM based on these tests:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1090#issuecomment-158117109
Merged to master, cannot merge to 4.6 as it would conflict when forward
merging. It will be in 4.7.
---
If your project is set up for it, you can reply to
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1084#issuecomment-158117634
I also agree we shouldn't keep unused files in the active tree. We already
got some PRs against files that were not in use and that is a waste of tim
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1089#issuecomment-158144546
Before merge, I want to retry the two failed tests and see if they work for
me.
---
If your project is set up for it, you can reply to this email and have
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1057#issuecomment-158314116
FYI: @DaanHoogland My tests show 29 test failures. I will retest to be
sure, but please hold until this is done.
---
If your project is set up for it, you can
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1094#issuecomment-158334828
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first). Screenshot is from an experimental Jenkins job (that runs the
same
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1089#issuecomment-158338131
@DaanHoogland the tests succeed for me:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true smoke
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1083#issuecomment-158341279
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first). Screenshot is from an experimental Jenkins job (that runs the
same
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/924#issuecomment-158348133
I tried rebasing against current master but I run into build issues. Tried
again but same thing.
@nnesic Could you try rebasing the PR against master
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/commit/3381154fafb7fa4f0a61d538f7c2550e48247787#commitcomment-14536914
In scripts/vm/systemvm/injectkeys.sh:
In scripts/vm/systemvm/injectkeys.sh on line 89:
This breaks CentOS7, see
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1058#issuecomment-158530260
I will run some tests on this branch and report back.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1057#issuecomment-158613419
@DaanHoogland @rafaelweingartner Much better now, just one failed test to
resolve:
Failed test: `TestVPCRouterOneNetwork`
```
Execute cmd
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1097#issuecomment-158613518
Pinging @milamberspace to have a look.
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1095#issuecomment-158613584
@borisroman Did you build the packages from this branch? Can you share some
details to show it works properly now?
---
If your project is set up for it, you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/977#issuecomment-158615300
Hi @wido can you review this again please? 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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1057#issuecomment-158624231
@DaanHoogland It could be a result of another failure. On master this test
passes, so maybe you can run just this test yourself and look at the details
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1057#issuecomment-158625241
:-)
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1075#issuecomment-158668313
LGTM, based on a set of tests that I run on this branch. Screenshot is from
an experimental Jenkins job (that runs the same tests I did manually until
now
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/969#issuecomment-158673227
Hi @rodrigo93 Thanks for rebasing, but please make sure you have one commit
(and get rid of the merge commit). Thanks!
---
If your project is set up for it
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/955#issuecomment-158673310
@SudharmaJain Any update on this? Please respond to the comments, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/944#issuecomment-158673838
Hi @pedro-martins I can run some tests but please remove the merge commit.
Thanks!
---
If your project is set up for it, you can reply to this email and have
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/943#issuecomment-158675017
Hi @rodrigo93 in the comments above it was suggested to change the commit
title, can you do that please? The commit title is the first line of the commit
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/926#issuecomment-158676063
@wido I see some comments above. Not sure if they are all addressed. Can
you please review this and let us know if you are OK with merging? I'll run
soms
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/886#issuecomment-158676481
@borisroman Can you have a look and resolve the conflict? When we find
another reviewer we can merge this in 4.7. Pinging @DaanHoogland @bhaisaab
---
If your
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/866#issuecomment-158676663
Pinging @wilderrodrigues to have a look.
@bvbharatk Is there any (integration) test to test this feature?
---
If your project is set up for it, you can
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1081#issuecomment-158678341
Thanks for the fix @nitin-maharana. Tried it and it works great! LGTM
https://cloud.githubusercontent.com/assets/1630096/11320287/64aad05a-9094-11e5
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1058#issuecomment-158752169
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first). Screenshot is from an experimental Jenkins job (that runs the
same
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1058#issuecomment-158752741
This was merged to 4.6, then to master as it is a security improvement that
applied cleanly.
@bhaisaab You may want to include this in the final 4.5
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1051#issuecomment-158752800
LGTM, just to be sure run all integration tests against this branch. They
all passed.
https://cloud.githubusercontent.com/assets/1630096/11323652
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1081#issuecomment-158752938
@DaanHoogland resizing the screen? I didn't test that.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/768#issuecomment-158753409
@abhinandanprateek @bhaisaab @jburwell Hi guys, I want to run soms tests to
see if we can get this in 4.7. As master is on 4.7.0-SNAPSHOT now, please also
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1019#issuecomment-158753796
Since the commits changed I run the tests again. Still LGTM.
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1103#issuecomment-158776432
Thanks @pdion891! I agree with all of them, except the systemvm template.
There is no reason to bump that version because there is no change yet. As soon
as we
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1103#issuecomment-158777294
@pdion891 See comments on
https://issues.apache.org/jira/browse/CLOUDSTACK-9077 for possible solution for
`/dev/loop0` issue.
---
If your project is set up
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1092#issuecomment-158791293
LGTM, all integration tests pass.
![screen shot 2015-11-22 at 20 24 54
pm](https://cloud.githubusercontent.com/assets/1630096/11325793/3c600dfe-9157
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1018#issuecomment-158792733
LGTM based on these tests:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1103#issuecomment-158895870
@bhaisaab Sure, that would be a good reason. But it's not ready so let's
bump when we merge that PR.
---
If your project is set up for it, you ca
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1102#issuecomment-158896104
LGTM, based on a set of tests that I run on this branch. Screenshot is from
an experimental Jenkins job (that runs the same tests I did manually until now
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/929#issuecomment-158897259
Run the tests again since master moved on quite a bit in the past weeks.
Still LGTM based on these tests:
```
nosetests --with-marvin
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1081#issuecomment-158898488
@DaanHoogland Is that what you mean.. testing it with the wider UI or..?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1049#issuecomment-158901032
LGTM based on these tests:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=true \
component
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1049#issuecomment-158901139
@wido @DaanHoogland @wilderrodrigues Did any of you run the unit test and
did they pass? If so, please paste output. After this, OK to merge?
---
If your
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1021#issuecomment-158901685
LGTM based on these tests (may not have tested the actual change):
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1007#issuecomment-158914722
Repeated the tests due to new commits:
LGTM based on these tests:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags
Github user remibergsma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1007#discussion_r45595780
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
---
@@ -3310,13 +3331,19 @@ public Vlan dedicatePublicIpRange(final
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1069#issuecomment-158915937
LGTM, built a cloud on this PR and clicked through the UI which worked fine.
![screen shot 2015-11-23 at 13 05
07](https://cloud.githubusercontent.com
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1106#issuecomment-158916248
Thanks @bhaisaab will test soon.
---
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
GitHub user remibergsma opened a pull request:
https://github.com/apache/cloudstack/pull/1109
CLOUDSTACK-9077 Fix injectkeys.sh to work on CentOS7
Fix regression from commit 3381154fafb7fa4f0a61d538f7c2550e48247787
The error seen on CentOS 7:
```
2015-11-20 21
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1103#issuecomment-159179258
@pdion891 @bhaisaab In about two weeks (Dec 7) we freeze for 4.7.0. That
would be a nice time to look back and see if the systemvm template needs to be
bumped
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1112#issuecomment-159179498
Pinging @pdion891 to have a look.
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1110#issuecomment-159267770
Guys, why didn't we get 2x LGTM on this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1113#issuecomment-159411260
What about 4.6?
---
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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack-www/pull/20#issuecomment-159526424
@bhaisaab what do you want to achieve? What's different with the shapeblue
packages? Why mentioning company names? It's highly confusing if
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack-www/pull/20#issuecomment-159530377
Stating 3rd party I agree with. I just don't want the confusion of having
different company names and people saying on the list they use "the
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack-docs-rn/pull/26#issuecomment-159561086
LGTM, aside for the already mentioned items. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1109#issuecomment-159564428
@pdion891 Thanks, please have a look.
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1094#issuecomment-159592539
@nvazquez Ai, there is a conflict now. Could you please have a look?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user remibergsma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1007#discussion_r46066286
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
---
@@ -3310,13 +3331,19 @@ public Vlan dedicatePublicIpRange(final
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/872#issuecomment-160189412
@jayapalu OK, no problem. FYI, we freeze on Dec 7.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1100#issuecomment-160190122
@alexandrelimassantana Can you rebase please and resolve the conflict?
Thanks!
---
If your project is set up for it, you can reply to this email and have your
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1133#issuecomment-160190800
Thanks @syed !
@pdion891 can you test this over the weekend? Then we can include it in
4.6.1.
---
If your project is set up for it, you can reply to this
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1133#issuecomment-160193249
@pdube Great!
---
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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/768#issuecomment-160282940
All integration tests pass, so that's good! Didn't look at the Quota
functionality itself yet.
![screen shot 2015-11-28 at 12 35 17
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1135#issuecomment-160283528
@pdube Nice one, will give it a try!
---
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
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1135#issuecomment-160287896
@pdube Had a look and you may also want to tweak these settings:
# Network garbage collection
network.gc.wait
network.gc.interval
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1106#issuecomment-160288024
LGTM, works fine and I was able to sort columns (example global settings).
![screen shot 2015-11-28 at 13 04 27
pm](https://cloud.githubusercontent.com
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1106#issuecomment-160288038
@bhaisaab had a busy week, so took a little longer ;-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user remibergsma commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1134#discussion_r46079415
--- Diff: test/integration/component/test_affinity_groups_projects.py ---
@@ -0,0 +1,1083 @@
+#!/usr/bin/env python
+# Licensed to the
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1134#issuecomment-160288372
@pdube Awesome you added all the tests! Thanks! Will have a run some tests
myself soon.
---
If your project is set up for it, you can reply to this email and
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-160290985
LGTM, haven't verified Debian packages though.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1101#issuecomment-160296390
@ustcweizhou LGTM, tested it and works as expected.
Minor comment: the first item in the list is empty (it will then keep the
same template), rest of
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1101#issuecomment-160296404
@bhaisaab @DaanHoogland Can you guys also review please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1081#issuecomment-160296533
@DaanHoogland This is already much better, when you have 30 port forwarding
rules the UI is unusable without this fix. When we want to make more
improvements
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1115#issuecomment-160298327
If we want this in 4.6, please don't merge this. When #1116 is merged in
4.6, it will be fwd merged to master as well.
---
If your project is set up f
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1133#issuecomment-160298443
@pdube Any news on your testing? :-)
---
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
901 - 1000 of 1079 matches
Mail list logo