[GitHub] [cloudstack] blueorangutan commented on issue #3323: Fixed: User not allowed to tag project created by him

[GitBox]

blueorangutan commented on issue #3323: Fixed: User not allowed to tag project 
created by him
URL: https://github.com/apache/cloudstack/pull/3323#issuecomment-492501406
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2736


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] blueorangutan commented on issue #3323: Fixed: User not allowed to tag project created by him

[GitBox]

blueorangutan commented on issue #3323: Fixed: User not allowed to tag project 
created by him
URL: https://github.com/apache/cloudstack/pull/3323#issuecomment-492498070
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] rhtyd commented on issue #3326: Bug fix for zone names not appearing in dashboard

[GitBox]

rhtyd commented on issue #3326: Bug fix for zone names not appearing in 
dashboard
URL: https://github.com/apache/cloudstack/pull/3326#issuecomment-492497842
 
 
   Purely UI change, screenshot and testing is OK, no explicit integration 
smoketests required.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] rhtyd commented on issue #3323: Fixed: User not allowed to tag project created by him

[GitBox]

rhtyd commented on issue #3323: Fixed: User not allowed to tag project created 
by him
URL: https://github.com/apache/cloudstack/pull/3323#issuecomment-492497973
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] rhtyd merged pull request #3326: Bug fix for zone names not appearing in dashboard

[GitBox]

rhtyd merged pull request #3326: Bug fix for zone names not appearing in 
dashboard
URL: https://github.com/apache/cloudstack/pull/3326
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] rhtyd closed issue #3314: Zone names missing in GUI dashboard in ACS411

[GitBox]

rhtyd closed issue #3314: Zone names missing in GUI dashboard in ACS411
URL: https://github.com/apache/cloudstack/issues/3314
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[cloudstack] branch master updated: ui: Bug fix for zone names not appearing in dashboard (#3326)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/master by this push:
 new 3e64285  ui: Bug fix for zone names not appearing in dashboard (#3326)
3e64285 is described below

commit 3e642852a7e3a58652118cfb28cd3699f1b15009
Author: Anurag Awasthi <43956255+anura...@users.noreply.github.com>
AuthorDate: Wed May 15 09:52:38 2019 +0530

ui: Bug fix for zone names not appearing in dashboard (#3326)

3 years ago when we moved from JSP this translate was not wrapped
in label tags and consequently the following span wasn't captured
when dashboard.js mapped the data.

Fixes: #3314
---
 ui/index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ui/index.html b/ui/index.html
index 9e49b36..b9d961d 100644
--- a/ui/index.html
+++ b/ui/index.html
@@ -1426,7 +1426,7 @@
 
 
 
-: 
+: 
 
 
 %

[GitHub] [cloudstack] GabrielBrascher commented on a change in pull request #3329: Fix: Migration target has no matching tags

[GitBox]

GabrielBrascher commented on a change in pull request #3329: Fix: Migration 
target has no matching tags
URL: https://github.com/apache/cloudstack/pull/3329#discussion_r283941164
 
 

 ##
 File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java
 ##
 @@ -2079,11 +2078,12 @@ public Volume migrateVolume(MigrateVolumeCmd cmd) {
 // OfflineVmwareMigration: check storage tags on disk(offering)s in 
comparison to destination storage pool
 // OfflineVmwareMigration: if no match return a proper error now
 DiskOfferingVO diskOffering = 
_diskOfferingDao.findById(vol.getDiskOfferingId());
-if(diskOffering.equals(null)) {
-throw new CloudRuntimeException("volume '" + vol.getUuid() +"', 
has no diskoffering. Migration target cannot be checked.");
+if (diskOffering.equals(null)) {
+throw new CloudRuntimeException("volume '" + vol.getUuid() + "', 
has no diskoffering. Migration target cannot be checked.");
 }
-if(! doesTargetStorageSupportDiskOffering(destPool, diskOffering)) {
-throw new CloudRuntimeException("Migration target has no matching 
tags for volume '" +vol.getName() + "(" + vol.getUuid() + ")'");
+if (!doesTargetStorageSupportDiskOffering(destPool, diskOffering)) {
+throw new CloudRuntimeException(String.format("Migration target 
pool [%s, tags:%s] has no matching tags for volume [%s, uuid:%s, tags:%s]", 
destPool.getName(),
 
 Review comment:
   Added some details in the log message.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] GabrielBrascher commented on a change in pull request #3329: Fix: Migration target has no matching tags

[GitBox]

GabrielBrascher commented on a change in pull request #3329: Fix: Migration 
target has no matching tags
URL: https://github.com/apache/cloudstack/pull/3329#discussion_r283941164
 
 

 ##
 File path: server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java
 ##
 @@ -2079,11 +2078,12 @@ public Volume migrateVolume(MigrateVolumeCmd cmd) {
 // OfflineVmwareMigration: check storage tags on disk(offering)s in 
comparison to destination storage pool
 // OfflineVmwareMigration: if no match return a proper error now
 DiskOfferingVO diskOffering = 
_diskOfferingDao.findById(vol.getDiskOfferingId());
-if(diskOffering.equals(null)) {
-throw new CloudRuntimeException("volume '" + vol.getUuid() +"', 
has no diskoffering. Migration target cannot be checked.");
+if (diskOffering.equals(null)) {
+throw new CloudRuntimeException("volume '" + vol.getUuid() + "', 
has no diskoffering. Migration target cannot be checked.");
 }
-if(! doesTargetStorageSupportDiskOffering(destPool, diskOffering)) {
-throw new CloudRuntimeException("Migration target has no matching 
tags for volume '" +vol.getName() + "(" + vol.getUuid() + ")'");
+if (!doesTargetStorageSupportDiskOffering(destPool, diskOffering)) {
+throw new CloudRuntimeException(String.format("Migration target 
pool [%s, tags:%s] has no matching tags for volume [%s, uuid:%s, tags:%s]", 
destPool.getName(),
 
 Review comment:
   Adding some details in the log message.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] GabrielBrascher opened a new pull request #3329: Fix: Migration target has no matching tags

[GitBox]

GabrielBrascher opened a new pull request #3329: Fix: Migration target has no 
matching tags
URL: https://github.com/apache/cloudstack/pull/3329
 
 
   ## Description
   
   The code prior to this commit was looking into storage tags at the table 
`storage_pool_details`. However, the table. It should select from 
`storage_pool_tags`, which contains the tags from each tagged storage.
   
   
   
   
   
   
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [X] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## Screenshots (if appropriate):
   
   ## How Has This Been Tested?
   
   
   
   Migrated a volume with a tag 'A' to a storage pool that also has the tag 
'A'. Resulted on the following exception:
   ```
   2019-05-10 17:03:16,343 ERROR [c.c.a.ApiAsyncJobDispatcher] 
(API-Job-Executor-9:ctx-1255172b job-16095) (logid:3f7e0f22) Unexpected 
exception while executing 
org.apache.cloudstack.api.command.admin.volume.MigrateVolumeCmdByAdmin
   com.cloud.utils.exception.CloudRuntimeException: Migration target has no 
matching tags for volume 'ROOT-2086(262d639d-a0b0-4f8d-9e86-01fb210ace8f)'
   at 
com.cloud.storage.VolumeApiServiceImpl.migrateVolume(VolumeApiServiceImpl.java:2086)
   at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
   at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
   at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
   at java.lang.reflect.Method.invoke(Method.java:498)
   ```
   After update with the proposed fix, the volume got migrated successfully.
   
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] webermaximilian92 edited a comment on issue #3328: Enhancement scss refactoring

[GitBox]

webermaximilian92 edited a comment on issue #3328: Enhancement scss refactoring
URL: https://github.com/apache/cloudstack/pull/3328#issuecomment-49057
 
 
   Hey,
   this PR may seem confusing in first moment.
   The most interesting file of those 86 is "_cloudstack3.css_"
   **All other files** are just the source to compile this final 13000 lines 
css-file now and may be ignored in this PR.
   Later we can use those new smaller scss-files to adjust styles more safety 
and faster.
   
   Max


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] webermaximilian92 edited a comment on issue #3328: Enhancement scss refactoring

[GitBox]

webermaximilian92 edited a comment on issue #3328: Enhancement scss refactoring
URL: https://github.com/apache/cloudstack/pull/3328#issuecomment-49057
 
 
   Hey,
   this PR may seem confusing in first moment.
   The most interesting file of those 86 is "_cloudstack3.css_"
   **All other files** are just the source to compile this final 13000 lines 
css-file now and may be ignored in this PR.
   Later we can use those new smaller scss-files to adjust styles more safty 
and faster.
   
   Max


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] webermaximilian92 commented on issue #3328: Enhancement scss refactoring

[GitBox]

webermaximilian92 commented on issue #3328: Enhancement scss refactoring
URL: https://github.com/apache/cloudstack/pull/3328#issuecomment-49057
 
 
   I know this PR may seem confusing to most of you first... 
   The only interesting file of those 86 is "_cloudstack3.css_"!
   **All other files** are just the source to build this final massive 13000 
lines css file now and may be ignored in this PR. 
   Later we can use those new smaller scss-files to adjust styles more safty 
and faster.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] DennisKonrad commented on issue #3319: Use IDE as the bus type for root disks and VIRTIO for data disks when platform emulator strings start with Windows* except Windows PV

[GitBox]

DennisKonrad commented on issue #3319: Use IDE as the bus type for root disks 
and VIRTIO for data disks when platform emulator strings start with Windows* 
except Windows PV
URL: https://github.com/apache/cloudstack/pull/3319#issuecomment-492214266
 
 
   @skattoju4 @svenvogel I will build this today and testing can happen the 
following days.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] skattoju4 commented on issue #3319: Use IDE as the bus type for root disks and VIRTIO for data disks when platform emulator strings start with Windows* except Windows PV

[GitBox]

skattoju4 commented on issue #3319: Use IDE as the bus type for root disks and 
VIRTIO for data disks when platform emulator strings start with Windows* except 
Windows PV
URL: https://github.com/apache/cloudstack/pull/3319#issuecomment-492211937
 
 
   yes, just the root disk will be ide now.
   
   On Tue., May 14, 2019, 8:09 a.m. Sven Vogel, 
   wrote:
   
   > @skattoju4  we will test it
   >
   > (one additional ide disk is possible before the error is triggered if i am
   > not mistaken)
   > unfortunately no. only one ide disk so what i know.
   >
   > okay there should not be a second ide disk right?
   >
   > —
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > 
,
   > or mute the thread
   > 

   > .
   >
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] svenvogel commented on issue #3319: Use IDE as the bus type for root disks and VIRTIO for data disks when platform emulator strings start with Windows* except Windows PV

[GitBox]

svenvogel commented on issue #3319: Use IDE as the bus type for root disks and 
VIRTIO for data disks when platform emulator strings start with Windows* except 
Windows PV
URL: https://github.com/apache/cloudstack/pull/3319#issuecomment-492210667
 
 
   @skattoju4 we will test it 
   > (one additional ide disk is possible before the error is triggered if i am 
not mistaken)
   unfortunately no. only one ide disk so what i know.
   
   okay there should not be a second ide disk right?
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[cloudstack] branch 4.12 updated (9488c6d -> 00ff536)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch 4.12
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


from 9488c6d  Merge remote-tracking branch 'origin/4.11' into 4.12
 add b60daf7  server: Fix exception while update domain resource count 
(#3204)
 add f9b61bc  orchestration: Allow VM that has never started to have 
volumes attached (#3276)
 add 9ff819d  systemvm: new qemu-guest-agent based patching for KVM (#3278)
 add 00ff536  Merge remote-tracking branch 'origin/4.11' into 4.12

No new revisions were added by this update.

Summary of changes:
 api/src/main/java/com/cloud/storage/Volume.java|   4 +-
 .../configuration/dao/ResourceCountDaoImpl.java|   5 +-
 .../storage/volume/VolumeServiceImpl.java  |   3 +
 .../kvm/resource/LibvirtComputingResource.java |  20 +--
 .../wrapper/LibvirtStartCommandWrapper.java|   7 +-
 .../kvm/resource/LibvirtDomainXMLParserTest.java   |  22 +---
 .../hypervisor/kvm/resource/LibvirtVMDefTest.java  |   2 +-
 scripts/vm/hypervisor/kvm/patch.sh |  80 
 scripts/vm/hypervisor/kvm/patchviasocket.py|  76 ---
 scripts/vm/hypervisor/kvm/test_patchviasocket.py   | 144 -
 .../com/cloud/storage/VolumeApiServiceImpl.java|  47 ---
 .../debian/opt/cloud/bin/setup/cloud-early-config  |  52 +++-
 systemvm/debian/root/.ssh/authorized_keys  |   1 -
 systemvm/pom.xml   |   7 -
 test/integration/smoke/test_volumes.py |  94 +++---
 .../scripts/configure_systemvm_services.sh |   2 +-
 tools/appliance/systemvmtemplate/template.json |   4 +-
 17 files changed, 229 insertions(+), 341 deletions(-)
 create mode 100755 scripts/vm/hypervisor/kvm/patch.sh
 delete mode 100755 scripts/vm/hypervisor/kvm/patchviasocket.py
 delete mode 100755 scripts/vm/hypervisor/kvm/test_patchviasocket.py
 delete mode 100644 systemvm/debian/root/.ssh/authorized_keys

[GitHub] [cloudstack] rhtyd commented on a change in pull request #3146: RIP Nuage Cloudstack Plugin

[GitBox]

rhtyd commented on a change in pull request #3146: RIP Nuage Cloudstack Plugin
URL: https://github.com/apache/cloudstack/pull/3146#discussion_r283715006
 
 

 ##
 File path: 
engine/schema/src/main/resources/META-INF/db/schema-41120to41200-cleanup.sql
 ##
 @@ -25,3 +25,7 @@ DROP TABLE IF EXISTS `cloud`.`iam_group_policy_map`;
 DROP TABLE IF EXISTS `cloud`.`iam_group`;
 DROP TABLE IF EXISTS `cloud`.`iam_policy_permission`;
 DROP TABLE IF EXISTS `cloud`.`iam_policy`;
+
+-- Cleanup Nuage VSP
+DELETE FROM `cloud`.`host` WHERE id in (SELECT vsp.host_id FROM 
`cloud`.`external_nuage_vsp_devices` vsp);
 
 Review comment:
   cc @DaanHoogland @krissterckx  - the upgrade path needs to be in 4.12->4.13. 
I'll fix that while forward merging 4.11->4.12->master.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[cloudstack] 02/02: Merge branch '4.12'

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 0700d91a685701bd25a8d1e5a9e46780e971
Merge: e11f7ee 00ff536
Author: Rohit Yadav 
AuthorDate: Tue May 14 15:13:57 2019 +0530

Merge branch '4.12'

- Fixes PR #3146 db cleanup to the correct 4.12->4.13 upgrade path
- Fixes failing unit test due to jdk specific changes after forward
  merging

Signed-off-by: Rohit Yadav 

 api/src/main/java/com/cloud/storage/Volume.java|   4 +-
 .../configuration/dao/ResourceCountDaoImpl.java|   5 +-
 .../META-INF/db/schema-41120to41200-cleanup.sql|   5 -
 .../META-INF/db/schema-41200to41300-cleanup.sql|   4 +
 .../storage/volume/VolumeServiceImpl.java  |   3 +
 .../kvm/resource/LibvirtComputingResource.java |  20 +--
 .../wrapper/LibvirtStartCommandWrapper.java|   7 +-
 .../kvm/resource/LibvirtDomainXMLParserTest.java   |  22 +---
 .../hypervisor/kvm/resource/LibvirtVMDefTest.java  |   2 +-
 scripts/vm/hypervisor/kvm/patch.sh |  80 
 scripts/vm/hypervisor/kvm/patchviasocket.py|  76 ---
 scripts/vm/hypervisor/kvm/test_patchviasocket.py   | 144 -
 .../com/cloud/storage/VolumeApiServiceImpl.java|  47 ---
 services/console-proxy/rdpconsole/pom.xml  |  13 ++
 .../debian/opt/cloud/bin/setup/cloud-early-config  |  52 +++-
 systemvm/debian/root/.ssh/authorized_keys  |   1 -
 systemvm/pom.xml   |   7 -
 test/integration/smoke/test_volumes.py |  94 +++---
 .../scripts/configure_systemvm_services.sh |   2 +-
 tools/appliance/systemvmtemplate/template.json |   4 +-
 20 files changed, 246 insertions(+), 346 deletions(-)

diff --cc 
engine/schema/src/main/resources/META-INF/db/schema-41200to41300-cleanup.sql
index 5702186,000..70c8d53
mode 100644,00..100644
--- 
a/engine/schema/src/main/resources/META-INF/db/schema-41200to41300-cleanup.sql
+++ 
b/engine/schema/src/main/resources/META-INF/db/schema-41200to41300-cleanup.sql
@@@ -1,21 -1,0 +1,25 @@@
 +-- Licensed to the Apache Software Foundation (ASF) under one
 +-- or more contributor license agreements.  See the NOTICE file
 +-- distributed with this work for additional information
 +-- regarding copyright ownership.  The ASF licenses this file
 +-- to you under the Apache License, Version 2.0 (the
 +-- "License"); you may not use this file except in compliance
 +-- with the License.  You may obtain a copy of the License at
 +--
 +--   http://www.apache.org/licenses/LICENSE-2.0
 +--
 +-- Unless required by applicable law or agreed to in writing,
 +-- software distributed under the License is distributed on an
 +-- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 +-- KIND, either express or implied.  See the License for the
 +-- specific language governing permissions and limitations
 +-- under the License.
 +
 +--;
 +-- Schema upgrade cleanup from 4.12.0.0 to 4.13.0.0
 +--;
 +
++-- Cleanup Nuage VSP
++DELETE FROM `cloud`.`host` WHERE id in (SELECT vsp.host_id FROM 
`cloud`.`external_nuage_vsp_devices` vsp);
++DROP TABLE IF EXISTS `cloud`.`external_nuage_vsp_devices`;
++DELETE FROM `cloud`.`role_permissions` WHERE rule = 
'issueNuageVspResourceRequest';
diff --cc services/console-proxy/rdpconsole/pom.xml
index 1b4a5ef,f0534f3..4c0e0df
--- a/services/console-proxy/rdpconsole/pom.xml
+++ b/services/console-proxy/rdpconsole/pom.xml
@@@ -49,4 -49,4 +49,17 @@@
  bcprov-jdk15on
  
  
++
++
++
++org.apache.maven.plugins
++maven-surefire-plugin
++
++
++rdpclient/MockServerTest.java
++
++
++
++
++
  

[cloudstack] branch master updated (e11f7ee -> 0700d91)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


from e11f7ee  RIP Nuage Cloudstack Plugin (#3146)
 add b60daf7  server: Fix exception while update domain resource count 
(#3204)
 add f9b61bc  orchestration: Allow VM that has never started to have 
volumes attached (#3276)
 add 9ff819d  systemvm: new qemu-guest-agent based patching for KVM (#3278)
 new 00ff536  Merge remote-tracking branch 'origin/4.11' into 4.12
 new 0700d91  Merge branch '4.12'

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 api/src/main/java/com/cloud/storage/Volume.java|   4 +-
 .../configuration/dao/ResourceCountDaoImpl.java|   5 +-
 .../META-INF/db/schema-41120to41200-cleanup.sql|   5 -
 .../META-INF/db/schema-41200to41300-cleanup.sql|   4 +
 .../storage/volume/VolumeServiceImpl.java  |   3 +
 .../kvm/resource/LibvirtComputingResource.java |  20 +--
 .../wrapper/LibvirtStartCommandWrapper.java|   7 +-
 .../kvm/resource/LibvirtDomainXMLParserTest.java   |  22 +---
 .../hypervisor/kvm/resource/LibvirtVMDefTest.java  |   2 +-
 scripts/vm/hypervisor/kvm/patch.sh |  80 
 scripts/vm/hypervisor/kvm/patchviasocket.py|  76 ---
 scripts/vm/hypervisor/kvm/test_patchviasocket.py   | 144 -
 .../com/cloud/storage/VolumeApiServiceImpl.java|  47 ---
 services/console-proxy/rdpconsole/pom.xml  |  13 ++
 .../debian/opt/cloud/bin/setup/cloud-early-config  |  52 +++-
 systemvm/debian/root/.ssh/authorized_keys  |   1 -
 systemvm/pom.xml   |   7 -
 test/integration/smoke/test_volumes.py |  94 +++---
 .../scripts/configure_systemvm_services.sh |   2 +-
 tools/appliance/systemvmtemplate/template.json |   4 +-
 20 files changed, 246 insertions(+), 346 deletions(-)
 create mode 100755 scripts/vm/hypervisor/kvm/patch.sh
 delete mode 100755 scripts/vm/hypervisor/kvm/patchviasocket.py
 delete mode 100755 scripts/vm/hypervisor/kvm/test_patchviasocket.py
 delete mode 100644 systemvm/debian/root/.ssh/authorized_keys

[GitHub] [cloudstack] DaanHoogland merged pull request #3146: RIP Nuage Cloudstack Plugin

[GitBox]

DaanHoogland merged pull request #3146: RIP Nuage Cloudstack Plugin
URL: https://github.com/apache/cloudstack/pull/3146
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [cloudstack] DaanHoogland commented on issue #3321: Cannot associate a public IP to a new isolated network (4.12.0.0)

[GitBox]

DaanHoogland commented on issue #3321: Cannot associate a public IP to a new 
isolated network (4.12.0.0)
URL: https://github.com/apache/cloudstack/issues/3321#issuecomment-492150325
 
 
   @svanharmelen i had a look at several solution paths and came across the old 
persistent flag. Is it feasible for you to use a network offering with the 
persistent flag on? It will be implemented even without instances.
   I realise this may come down to terraform's requirements, but if it is only 
a test-bed requirement you have, this may work around the issue.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services