Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@jburwell a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
r
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@blueorangutan test centos7 xenserver-65sp1
---
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 fe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Trillian test result (tid-517)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31158 seconds
Marvin logs:
https://github.com/blueoranguta
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy I see failures in the `test_router_dhcp_opts` test case. Was
this PR rebased to get the fix from #1801? If not, I would like to rebase and
re-run the tests. Otherwise, we may h
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1435
@pdion891 this
[blog](https://ariejan.net/2011/07/05/git-squash-your-latests-commits-into-one/)
is good description of squash process.
---
If your project is set up for it, you can reply to
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1804#discussion_r90476049
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
---
@@ -3092,8 +3092,12 @@ public Vlan createVlanAndPublicIpRange(final long
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1799
@rhtyd the Travis build failed due a timeout on one of the workers. Could
you please do a force push to trigger a new build?
---
If your project is set up for it, you can reply to this email
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@jburwell This PR is not rebased to fix from #1801. This PR is confined to
test_privategw_acl.py, so no scope of any regression out side of it.
---
If your project is set up for it, you ca
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy agreed that it is extremely unlikely that the code change
impacts that test case. However, we know that a failure to cleanup between
tests can cause failures when they are run to
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1773#discussion_r90481980
--- Diff:
api/src/org/apache/cloudstack/api/command/user/template/DeleteTemplateCmd.java
---
@@ -52,6 +52,9 @@
@Parameter(name = ApiConstants.
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1773#discussion_r90480678
--- Diff: server/src/com/cloud/template/TemplateManagerImpl.java ---
@@ -1176,6 +1176,23 @@ public boolean deleteTemplate(DeleteTemplateCmd cmd)
{
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@blueorangutan package
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1776
@priyankparihar could you please provide further explanation as to how this
fix addresses the issue of `concurrent.snapshots.threshold.perhost` not being
validated?
Also, is there an e
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/873
Trillian test result (tid-519)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27196 seconds
Marvin logs:
https://github.com/blueorangutan
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1800
Thanks, @rhtyd I've been trying to get test_snapshots.py to fail in my
environment, but haven't been able to.
I'll let @syed answer your question, Rohit, since that is a change he
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1786#discussion_r90487461
--- Diff:
plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
---
@@ -260,7 +264,7 @@ public boolean applyLBRules(Ne
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Packaging result: âcentos6 âcentos7 âdebian. JID-299
---
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 d
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1763
@rhtyd can you investigate why the fix for this issue in 4.5 was not pulled
forward? For traceability purposes, it would be preferable to forward merge
the fix than commit a new version of it.
I'll take a look at this. It's been some moths but afaik I added that
extension because previously that code wasn't working. I'll take a look in
detail and let you guys know
On Thu, Dec 1, 2016 at 11:34 mike-tutkowski wrote:
> Github user mike-tutkowski commented on the issue:
>
> https://git
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1799
---
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 issue:
https://github.com/apache/cloudstack/pull/1799
@blueorangutan package
---
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
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1799
CLOUDSTACK-9632: Upgrade bouncy castle to version 1.55
- Upgrades Maven dependency version to v1.55
- Fixes bountycastle usages and issues
- Adds timeout to jetty/annotation scanning
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Thanks @jburwell I'll get this re-tested against both Travis and Trillian.
---
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 p
Github user sudhansu7 commented on the issue:
https://github.com/apache/cloudstack/pull/1763
@rhtyd @jburwell
I think the fix for CLOUDSTACK-9376 will not address below scenario. I will
test this and confirm.
1. templatefilter=all or isofilter=all is applicable only
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Packaging result: âcentos6 âcentos7 âdebian. JID-300
---
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 d
Github user sgoeminn commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90509963
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
---
@@ -387,41 +400,39 @@ public boolean canEnableIndiv
Github user sgoeminn commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90510516
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
---
@@ -387,41 +400,39 @@ public boolean canEnableIndiv
Github user sgoeminn commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90510701
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/agent/api/manager/CleanUpDomainCommand.java
---
@@ -0,0 +1,63 @@
+//
+// Licensed to
GitHub user sudhansu7 opened a pull request:
https://github.com/apache/cloudstack/pull/1805
CLOUDSTACK-9637: Template create from snapshot does not populate vm_tâ¦
**ISSUE**
Template create from snapshot does not populate vm_template_details
**REPR
Github user sgoeminn commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90510869
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/agent/api/manager/UpdateNuageVspDeviceCommand.java
---
@@ -0,0 +1,43 @@
+//
+// Lice
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Packaging result: âcentos6 âcentos7 âdebian. JID-301
---
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 d
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy @rhtyd can you investigate the Travis failures?
---
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 doe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1799
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@jburwell a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7
mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@blueorangutan test matrix
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@jburwell the issue was with an intermittent test that sometimes fail when
clocks get skewed in virtualized environment: `
HypervisorUtilsTest.checkVolumeFileForActivityTest:68 Didn't block long
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/873
LGTM. Travis failure was due to an intermittent issue with oobm test, which
has been fixed now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/873
@abhinandanprateek @murali-reddy @jburwell this may be useful for 4.9/lts,
would you like to review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@abhinandanprateek could you please rebase this PR to pick up the fixes to
the broken tests? I want to make sure that this PR doesn't introduce any
side-effects that break the test runs.
---
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Coupled with @rhtyd's explanation, we can merge this PR if the current
blueorganutan run comes up clean.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user sgoeminn commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1579#discussion_r90520113
--- Diff:
plugins/network-elements/nuage-vsp/src/com/cloud/network/resource/NuageVspResourceConfiguration.java
---
@@ -0,0 +1,310 @@
+//
+// Li
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/828
Trillian test result (tid-520)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 6
Total time taken: 38128 seconds
Marvin logs:
https://github.com/blueorangutan
Github user sureshanaparti closed the pull request at:
https://github.com/apache/cloudstack/pull/1248
---
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 fe
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1248
This PR is no longer valid since #1257 replaces this. Closing...
---
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 sureshanaparti commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1257#discussion_r90532049
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -138,6 +142,8 @@
@Inject
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1257
Addressed all the changes suggested and rebased against latest master.
- Used CollectionUtils.isEmpty() as suggested.
---
If your project is set up for it, you can reply to this email
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1257
@blueorangutan test centos6 vmware-55u3
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Packaging result: âcentos6 âcentos7 âdebian. JID-302
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Trillian test result (tid-525)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6
Total time taken: 34869 seconds
Marvin logs:
https://github.com/blueoran
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-533)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35604 seconds
Marvin logs:
https://github.com/blueoran
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1659
Trillian test result (tid-532)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35829 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Trillian test result (tid-536)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 25203 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Trillian test result (tid-534)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26155 seconds
Marvin logs:
https://github.com/blueoranguta
Github user nitin-maharana commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1804#discussion_r90584766
--- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
---
@@ -3092,8 +3092,12 @@ public Vlan createVlanAndPublicIpRange(final
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1802
Trillian test result (tid-537)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 34106 seconds
Marvin logs:
https://github.com/blueoranguta
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/828
I looked at the failure, all of them are environment related or known
intermittent failures. I'll proceed with merging this. LGTM.
---
If your project is set up for it, you can reply to this email
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/828
---
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 reopened a pull request:
https://github.com/apache/cloudstack/pull/1799
CLOUDSTACK-9632: Upgrade bouncy castle to version 1.55
- Upgrades Maven dependency version to v1.55
- Fixes bountycastle usages and issues
- Adds timeout to jetty/annotation scanning
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1806
travis: cleanup apt before installing packages
This aims at fixing recent Travis failures.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sha
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1799
---
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 issue:
https://github.com/apache/cloudstack/pull/1566
LGTM. Thanks @ProjectMoon
@blueorangutan package
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1566
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1622
Thanks @ProjectMoon
@blueorangutan package
---
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 f
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1622
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1618
Thanks @ProjectMoon
@blueorangutan package
---
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 f
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1579
Travis is failing due to an env issue in their VMs, I'm investigating it
with #1806
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1618
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1257
@sureshanaparti sorry, this is a restricted command to avoid resource abuse
issues.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
repl
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1257
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/873
Test LGTM.
---
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 th
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/873#discussion_r90589240
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -1746,10 +1747,10 @@ public boolean
storagePoolHasEnoughSpace(List volumes, Storage
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/873#discussion_r90589284
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -1719,6 +1719,7 @@ public boolean storagePoolHasEnoughSpace(List
volumes, StoragePo
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1802
@murali-reddy can you have a look at why private_gw failed again for vmware?
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Tests look good so far, I'm waiting for a vmware specific test run to
complete. Some failures in kvm, xen are known intermittent failures.
---
If your project is set up for it, you can reply to t
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1797
@sudhansu7 could you please either add or update an existing a Marvin test
case to verify this change?
Also, this change seems like it would be useful for LTS users. Could you
please
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@jburwell rebased.
---
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 a
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1765#discussion_r90590680
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java
---
@@ -100,36 +100,25 @@ protected
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@blueorangutan package
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1765
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1800
Thanks @syed I'll go ahead and merge this, as the changes confirm. I'm okay
that moving (4.10+) fwd we've `.vhd` extension in snapshot files.
---
If your project is set up for it, you can reply t
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1800
---
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 issue:
https://github.com/apache/cloudstack/pull/1794
@karuturi ping
---
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
All,
I've sent a PR that will upgrade bountycastle dependency to the latest version
[1]. In terms of security, an upgrade is necessary though it would also require
for users (who are upgrading to 4.9.1.0, 4.10.0.0 or later) to destroy old
systemvms such as CPVM and SSVM so the agents that will
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1805
Nice fix, @sudhansu7 this looks useful, can you change the base branch for
the PR to 4.9, rebase your PR branch against 4.9?
---
If your project is set up for it, you can reply to this email and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1804
@nitin-maharana this looks useful, can you change the base branch for the
PR to 4.9, rebase your PR branch against 4.9? Can you add a marvin test for
this?
---
If your project is set up for it,
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1803
---
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 issue:
https://github.com/apache/cloudstack/pull/1798
@marcaurele one of the component tests have failed, please have a look
and/or push -f to rekick Travis.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1796
@sudhansu7 this may be useful, can you at least change the PR's base branch
to 4.9?
---
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1795
@blueorangutan package
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1741
@swill @jayapalu thanks for your work, can you rebase against latest
master, fix the conflicts. Also use the JIRA id in the git commit summary.
---
If your project is set up for it, you can repl
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1765#discussion_r90594670
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java
---
@@ -100,36 +100,25 @
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r90594621
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){
if(downlo
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1799
Test LGTM. @abhinandanprateek @murali-reddy can you give a final
review/lgtm?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1748
@mike-tutkowski you please fix the git commit summary (not the PR summary)
in the requested format? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1764
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1764
Thanks @marcaurele
@blueorangutan package
---
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 fe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1765
Test lgtm, based on the test results I'm seeing no new failures.
---
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 doe
101 - 200 of 214 matches
Mail list logo
