Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/897
---
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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-153125931
Hi @nitin-maharana hmm all those numbers ;-) sorry dude!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user nitin-maharana commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-153093925
Hi @remibergsma, I tried rebasing with current master. There is no conflict
as such.
I think you were trying to merge the PR #987, But this PR is
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-152993596
manually verified the fix.
Before the fix:
on list networks page, I see three api calls
1. list zones
2. list networks with SG groups
3. list
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-152995832
@remibergsma 2 LGTM's so it can be merged at your discretion.
---
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/897#issuecomment-153008102
This PR has a conflict now, can someone resolve it?
```
From https://github.com/apache/cloudstack
* [new ref] refs/pull/987/head ->
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-152214761
@nitin-maharana I looked at this, seems good. What is the best way to test
this or see the difference?
---
If your project is set up for it, you can reply to
Github user sudhansu7 commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-152273822
@remibergsma @runseb
I have below performance analysis, which I did some time back. I trying to
find out the dbdump to do a real test. But below analysis
Github user nitin-maharana commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-152412317
Thanks @sudhansu7. I had also the similar idea. @remibergsma @runseb please
look at the comments of sudhansu. Thanks.
---
If your project is set up for it,
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-147089728
LGTM (reviewed the changes, did not test it though)
---
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 runseb commented on the pull request:
https://github.com/apache/cloudstack/pull/897#issuecomment-145662988
thanks @nitin-maharana maybe we should have a small performance study to
show that it actually speeds things up
---
If your project is set up for it, you can reply
GitHub user nitin-maharana opened a pull request:
https://github.com/apache/cloudstack/pull/897
CLOUDSTACK-8919: Slow UI response while loading the list of networks in
network tab.
Instead of searching for each network, now it is searching for each zone.
For basic zone, it will
12 matches
Mail list logo