Github user pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1581
---
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 pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1581
@kishankavala I think that the ultimate fix will be in the VR. However, the
inversion of the list is fixed with this patch, and does not require a VR
update. This is a good enough fix for now, as
GitHub user pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1609
[CLOUDSTACK-9430] Added fix for adding/editing Network ACL rule ordering
BUG: https://issues.apache.org/jira/browse/CLOUDSTACK-9430
The issue occurred because all of the ACL rules get
Github user pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1609
---
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 pdube reopened a pull request:
https://github.com/apache/cloudstack/pull/1609
[CLOUDSTACK-9430] Added fix for adding/editing Network ACL rule ordering
BUG: https://issues.apache.org/jira/browse/CLOUDSTACK-9430
The issue occurred because all of the ACL rules get
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1609
The issue was with the teardown of a snapshots 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
GitHub user pdube reopened a pull request:
https://github.com/apache/cloudstack/pull/1609
[CLOUDSTACK-9430] Added fix for adding/editing Network ACL rule ordering
BUG: https://issues.apache.org/jira/browse/CLOUDSTACK-9430
The issue occurred because all of the ACL rules get
Github user pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1609
---
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 pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1609
Excellent. @leprechau were you able to test the fix?
---
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 pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1611#discussion_r71037734
--- Diff: tools/marvin/marvin/deployDataCenter.py ---
@@ -224,6 +225,16 @@ def createClusters(self, clusters, zoneId, podId,
vmwareDc=None
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1611
Nice. LGTM, code reviewed
---
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 pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1616
Added missing rules on router config, fixed ordering of multiple ruleâ¦
â¦s, removed duplicate rules, added fix for network stats, added a check
for b64 decoding (to pad incorrect b64). Also
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1445#issuecomment-199858341
Built and ran the unit tests. LGTM
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration
SUCCESS [ 1.106 s]
[INFO] Apache CloudStack
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203434250
Hi @GabrielBrascher
Travis noted:
Failed during checkstyle execution: There are 1 checkstyle errors.
Have you run *mvn validate* on your code
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1456#issuecomment-203437320
Hi @hogniRunar,
The code change LGTM. I verified for references to the function, but did
not test it. Could you create a JIRA ticket for it please? Could you
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1453#issuecomment-203469470
I built, ran the unit tests and ran the server with the simulator. Thanks
for cleaning up! LGTM
[INFO
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026583
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -223,10 +226,7 @@
private static ExecutorService s_executor = new ThreadPoolExecutor(10
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026830
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -1340,37 +1340,31 @@ public String getSerializedApiError(final
ServerApiException ex, final Map
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026874
--- Diff: server/src/com/cloud/api/ApiServlet.java ---
@@ -156,7 +158,7 @@ void processRequestInContext(final HttpServletRequest
req, final
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207490720
Built the package LGTM after review
[INFO]
[INFO] Reactor Summary:
[INFO
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1479#issuecomment-207537198
Simply added a log warning. Built it LGTM
[INFO]
[INFO] Reactor Summary:
[INFO
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1445#issuecomment-209017286
@swill this looks ready, no?
---
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 pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1445#issuecomment-209023846
@rafaelweingartner @swill The only thing changed was the unit tests, which
I ran. And the only code change he did was to remove useless variables. @swill
also gave
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1445#issuecomment-209026629
Just trying to help get things merged :)
---
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 pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1491#discussion_r59644306
--- Diff: server/src/com/cloud/user/AccountManagerImpl.java ---
@@ -761,6 +774,17 @@ protected boolean cleanupAccount(AccountVO account,
long callerUserId
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1491#discussion_r59644462
--- Diff: server/src/com/cloud/user/AccountManagerImpl.java ---
@@ -761,6 +774,17 @@ protected boolean cleanupAccount(AccountVO account,
long callerUserId
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730081
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730334
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59730927
--- Diff: api/src/org/apache/cloudstack/acl/RoleService.java ---
@@ -0,0 +1,43 @@
+// Licensed to the Apache Software Foundation (ASF) under one
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59777518
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1489#discussion_r59812425
--- Diff: api/src/org/apache/cloudstack/acl/RoleType.java ---
@@ -16,18 +16,90 @@
// under the License.
package org.apache.cloudstack.acl
GitHub user pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1505
CLOUDSTACK-9172 Added cross zones check to delete template and iso
Added a check to ignore the zoneid, in the delete template UI, if the
template is cross zones.
reference : CLOUDSTACK
GitHub user pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1504
CLOUDSTACK-9172 Added cross zones check to delete template and iso
Added a check to ignore the zoneid, in the delete template UI, if the
template is cross zones.
reference : CLOUDSTACK
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1504#issuecomment-212587181
Pushing towards 4.7
---
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 pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1504
---
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 pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1505#issuecomment-212950771
Screen shot from
[before](https://issues.apache.org/jira/secure/attachment/1250/Screen%20Shot%202015-12-15%20at%2015.32.12.png)
the fix
After
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1476#issuecomment-213116347
Hey @remibergsma just noticed that you had a PR open for this. I fixed it
as well with #1505 so we might be able to close this one
---
If your project is set up
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1477#issuecomment-213822378
LGTM. Do you think we should translate this?
---
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 pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1820
CLOUDSTACK-9656 Preventing autoboxing NPE in Usage by setting a default
role when not found
https://issues.apache.org/jira/browse/CLOUDSTACK-9656
This is a workaround to avoid NPE when
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1820
Ok will do, 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 have this feature
enabled and wishes so
Github user pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1820
---
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 pdube reopened a pull request:
https://github.com/apache/cloudstack/pull/1820
CLOUDSTACK-9656 Preventing autoboxing NPE in Usage by setting a default
role when not found
https://issues.apache.org/jira/browse/CLOUDSTACK-9656
This is a workaround to avoid NPE
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1820
@rhtyd Changed the PR to be against 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. If your project does not have this feature
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1820
@rhtyd Thanks for the speedy responses
---
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 pdube opened a pull request:
https://github.com/apache/cloudstack/pull/1830
CLOUDSTACK-9677: Adding storage policy support for swift as secondaryâ¦
⦠storage
Added support to use storage policies with swift
You can merge this pull request into a Git
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1858
Happy holidays!
---
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 pdube reopened a pull request:
https://github.com/apache/cloudstack/pull/1830
CLOUDSTACK-9677: Adding storage policy support for swift as secondaryâ¦
⦠storage
Added support to use storage policies with swift
You can merge this pull request into a Git
Github user pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1830
@syed Think you could take a look? 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 have this
Github user pdube closed the pull request at:
https://github.com/apache/cloudstack/pull/1830
---
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 pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1830
Oops, closed by accident
---
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 pdube commented on the issue:
https://github.com/apache/cloudstack/pull/1829
@rhtyd @koushik-das definitely a blocker IMHO. Being unable to start/reboot
a VM that has 3 volumes is unacceptable in a public cloud scenario
---
If your project is set up for it, you can reply
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r98259432
--- Diff:
plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixRequestWrapperTest.java
---
@@ -436,7 +436,7
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r98259959
--- Diff: server/src/com/cloud/vm/UserVmManagerImpl.java ---
@@ -3520,27 +3520,17 @@ public UserVmVO doInTransaction(TransactionStatus
status) throws
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r98259395
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixResizeVolumeCommandWrapper.java
---
@@ -48,6
101 - 154 of 154 matches
Mail list logo
