Re: [PR] CKS: make clustertype optional to keep backwards compatibility [cloudstack]

[via GitHub]

DaanHoogland commented on PR #8180:
URL: https://github.com/apache/cloudstack/pull/8180#issuecomment-1794261249

   @blueorangutan test


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1794266214

   @DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7344:
URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1794266295

   @DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Make authentication request parameter order to be deterministic [cloudstack]

[via GitHub]

blueorangutan commented on PR #8185:
URL: https://github.com/apache/cloudstack/pull/8185#issuecomment-1794266356

   @shwstppr a [SL] Jenkins job has been kicked to build packages. It will be 
bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] CKS: make clustertype optional to keep backwards compatibility [cloudstack]

[via GitHub]

blueorangutan commented on PR #8180:
URL: https://github.com/apache/cloudstack/pull/8180#issuecomment-1794266165

   @DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) 
has been kicked to run smoke tests


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1794269897

   Reprt was not uploaded:
   
   smoke tests completed. 116 look OK, 1 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_02_trigger_shutdown | `Failure` | 336.81 | test_safe_shutdown.py
   
   ```
   FAIL: test_02_trigger_shutdown 
(tests.smoke.test_safe_shutdown.TestSafeShutdown)
   --
   Traceback (most recent call last):
 File "/marvin/tests/smoke/test_safe_shutdown.py", line 121, in 
test_02_trigger_shutdown
   self.startServer()
 File "/marvin/tests/smoke/test_safe_shutdown.py", line 82, in startServer
   return self.fail("Management server did not come up, failing")
   AssertionError: Management server did not come up, failing
   ```
   
   Does not have anything to do with this PR, it seems


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Fixes:#7837: Add isolationMethods and vlan to TrafficTypeResponse [cloudstack]

[via GitHub]

DaanHoogland commented on code in PR #8151:
URL: https://github.com/apache/cloudstack/pull/8151#discussion_r1382906864


##
test/integration/smoke/test_network_traffic_type_api.py:
##
@@ -0,0 +1,50 @@
+from marvin.cloudstackTestCase import cloudstackTestCase

Review Comment:
   this will need a license
   ```suggestion
   # 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.
   
   # Import Local Modules
   from marvin.cloudstackTestCase import cloudstackTestCase
   ```



##
test/integration/smoke/test_network_traffic_type_api.py:
##
@@ -0,0 +1,50 @@
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.base import (
+TrafficType,
+PhysicalNetwork,
+Zone,
+)
+from nose.plugins.attrib import attr
+
+
+class TestTrafficTypeApi(cloudstackTestCase):
+@classmethod
+def setUpClass(cls):
+cls.test_client = super(TestTrafficTypeApi, cls).getClsTestClient()
+cls.api_client = cls.testClient.getApiClient()
+cls.services = cls.testClient.getParsedTestDataConfig()
+cls._cleanup = []
+cls.services["advanced_sg"]["zone"]["name"] = "TestTrafficTypeApi-zone"
+cls.zone = Zone.create(cls.api_client, 
cls.services["advanced_sg"]["zone"])
+cls._cleanup.append(cls.zone)
+
+cls.physical_network = PhysicalNetwork.create(
+cls.api_client,
+cls.services["l2-network"],
+isolationmethods="VLAN",
+zoneid=cls.zone.id,
+)
+cls._cleanup.append(cls.physical_network)

Review Comment:
   this will need a 
   ```suggestion
   cls._cleanup.append(cls.physical_network)
   
   @classmethod
   def tearDownClass(cls):
   super(TestTrafficTypeApi, cls).tearDownClass()
   
   ```
   to work



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Create global configuration to allow changing the default nic adapter for user VMs in VMware [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7954:
URL: https://github.com/apache/cloudstack/pull/7954#issuecomment-1794285091

   @blueorangutan test alma8 vmware-70u3 keepEnv


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Create global configuration to allow changing the default nic adapter for user VMs in VMware [cloudstack]

[via GitHub]

blueorangutan commented on PR #7954:
URL: https://github.com/apache/cloudstack/pull/7954#issuecomment-1794287724

   @DaanHoogland a [SL] Trillian-Jenkins test job (alma8 mgmt + vmware-70u3) 
has been kicked to run smoke tests


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

DaanHoogland commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1794291710

   Ah, this has to do with certificates. Uploading from local will fail this 
way. The trick is to try an
   
   -  upload while having the debugger/network log open in your browser
   - get the actual upload url
   - open another window
   - go to the base url of the upload
   - accept the server cert
   - go back and try to upload
   
   let me know if this works @rongage 
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] edge-zone,kvm,iso,cks: allow k8s deployment with direct-download iso [cloudstack]

[via GitHub]

kiranchavala commented on PR #8142:
URL: https://github.com/apache/cloudstack/pull/8142#issuecomment-1794291935

   @shwstppr  Thanks 
   
   functionality wise it's working fine , was able to spin up k8s cluster on 
edge zones  and launch vm's from iso 


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Externalize KVM HA heartbeat frequency [cloudstack]

[via GitHub]

DaanHoogland commented on PR #6892:
URL: https://github.com/apache/cloudstack/pull/6892#issuecomment-1794293836

   ai, conflicts, Can you have a look @GaOrtiga (cc @stephankruggg )


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Make authentication request parameter order to be deterministic [cloudstack]

[via GitHub]

weizhouapache commented on PR #8185:
URL: https://github.com/apache/cloudstack/pull/8185#issuecomment-1794326008

   > Nicira is likely not support, unmaintained - are you a Nicira user 
@rRajivramachandran - could you advise if there's a reason around the PR?
   
   we could remove the plugin in the near future
   
   this PR is not useful, but we can merge.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

weizhouapache commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1794331843

   > Reprt was not uploaded:
   > 
   > smoke tests completed. 116 look OK, 1 have errors, 0 did not run Only 
failed and skipped tests results shown below:
   > 
   > Test   Result  Time (s)Test File
   > test_02_trigger_shutdown   `Failure`   336.81  test_safe_shutdown.py
   > ```
   > FAIL: test_02_trigger_shutdown 
(tests.smoke.test_safe_shutdown.TestSafeShutdown)
   > --
   > Traceback (most recent call last):
   >   File "/marvin/tests/smoke/test_safe_shutdown.py", line 121, in 
test_02_trigger_shutdown
   > self.startServer()
   >   File "/marvin/tests/smoke/test_safe_shutdown.py", line 82, in startServer
   > return self.fail("Management server did not come up, failing")
   > AssertionError: Management server did not come up, failing
   > ```
   > 
   > Does not have anything to do with this PR, it seems
   
   @DaanHoogland 
   agree test failure is not related. fresh installation looks good.
   
   This PR have many database changes, which require intensive upgrade tests 
(from various older versions) IMHO.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Make authentication request parameter order to be deterministic [cloudstack]

[via GitHub]

DaanHoogland merged PR #8185:
URL: https://github.com/apache/cloudstack/pull/8185


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Make authentication request parameter order to be deterministic [cloudstack]

[via GitHub]

DaanHoogland commented on PR #8185:
URL: https://github.com/apache/cloudstack/pull/8185#issuecomment-1794342092

   > > Nicira is likely not support, unmaintained - are you a Nicira user 
@rRajivramachandran - could you advise if there's a reason around the PR?
   > 
   > we could remove the plugin in the near future
   > 
   > this PR is not useful, but we can merge.
   
   If it helps @rRajivramachandran , ... I think this is not bothering anyone 
else


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

(cloudstack) branch main updated: Make authentication request parameter order to be deterministic (#8185)

[dahn]

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

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


The following commit(s) were added to refs/heads/main by this push:
 new e9b24b6c326 Make authentication request parameter order to be 
deterministic (#8185)
e9b24b6c326 is described below

commit e9b24b6c3266f14a85f2db86fb5d824539206097
Author: rRajivramachandran 
AuthorDate: Mon Nov 6 02:53:49 2023 -0600

Make authentication request parameter order to be deterministic (#8185)
---
 .../src/main/java/com/cloud/network/nicira/NiciraRestClient.java  | 4 ++--
 .../src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java  | 4 ++--
 utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java   | 3 ++-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git 
a/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
 
b/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
index a98480b11b3..aa428b08dc3 100644
--- 
a/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
+++ 
b/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
@@ -20,7 +20,7 @@
 package com.cloud.network.nicira;
 
 import java.io.IOException;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 import org.apache.http.HttpEntity;
@@ -120,7 +120,7 @@ public class NiciraRestClient extends BasicRestClient {
 }
 
 private HttpUriRequest createAuthenticationRequest() {
-final Map parameters = new HashMap<>();
+final Map parameters = new LinkedHashMap<>();
 parameters.put("username", username);
 parameters.put("password", password);
 return HttpUriRequestBuilder.create()
diff --git 
a/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
 
b/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
index 75b751c8ae1..bae11c7bfa2 100644
--- 
a/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
+++ 
b/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
@@ -32,7 +32,7 @@ import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 import org.apache.http.HttpHost;
@@ -70,7 +70,7 @@ public class NiciraRestClientTest {
 private static final StatusLine HTTP_200_STATUSLINE = new 
BasicStatusLine(new ProtocolVersion(HTTPS, 1, 1), 200, "OK");
 private static final StatusLine HTTP_401_STATUSLINE = new 
BasicStatusLine(new ProtocolVersion(HTTPS, 1, 1), 401, "Unauthorized");
 
-private static final Map loginParameters = new 
HashMap();
+private static final Map loginParameters = new 
LinkedHashMap();
 private static HttpUriRequest request;
 private static HttpUriRequest loginRequest;
 private final CloseableHttpClient httpClient = 
mock(CloseableHttpClient.class);
diff --git 
a/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java 
b/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
index 47ac06af111..52039a92ede 100644
--- a/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
+++ b/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
@@ -22,6 +22,7 @@ package com.cloud.utils.rest;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Map.Entry;
 
@@ -47,7 +48,7 @@ public class HttpUriRequestBuilder {
 private String path;
 private Optional jsonPayload = ABSENT;
 private final Map parameters = new HashMap();
-private final Map methodParameters = new HashMap();
+private final Map methodParameters = new 
LinkedHashMap();
 
 private HttpUriRequestBuilder() {
 

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1794347303

   > This PR have many database changes, which require intensive upgrade tests 
(from various older versions) IMHO.
   
   ok, I'll do a simple upgrade test before merging. We'll have to test the RC 
intensively  for upgrade anyway.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

sonarcloud[bot] commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1794351389

   Kudos, SonarCloud Quality Gate passed!    [![Quality Gate 
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
 'Quality Gate 
passed')](https://sonarcloud.io/dashboard?id=apache_cloudstack&pullRequest=7345)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=BUG)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7345&resolved=false&types=SECURITY_HOTSPOT)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7345&resolved=false&types=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7345&resolved=false&types=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7345&resolved=false&types=CODE_SMELL)
   
   [![No Coverage 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png
 'No Coverage 
information')](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7345&metric=coverage&view=list)
 No Coverage information  
   [![No Duplication 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/NoDuplicationInfo-16px.png
 'No Duplication 
information')](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7345&metric=duplicated_lines_density&view=list)
 No Duplication information
   
   
![warning](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/message_warning-16px.png
 'warning') The version of Java (11.0.21) you have used to run this analysis is 
deprecated and we will stop accepting it soon. Please update to at least Java 
17.
   Read more [here](https://docs.sonarcloud.io/appendices/scanner-environment/)
   
   
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1794365174

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7641


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7344:
URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1794365179

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7642


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Make authentication request parameter order to be deterministic [cloudstack]

[via GitHub]

blueorangutan commented on PR #8185:
URL: https://github.com/apache/cloudstack/pull/8185#issuecomment-1794365181

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7643


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1794385354

   @blueorangutan test matrix


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

(cloudstack) branch dummy updated (95a61c0d996 -> c28d51a0ac9)

[dahn]

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

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


omit 95a61c0d996 do not merge
 add e9b24b6c326 Make authentication request parameter order to be 
deterministic (#8185)
 add c28d51a0ac9 do not merge

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (95a61c0d996)
\
 N -- N -- N   refs/heads/dummy (c28d51a0ac9)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .../src/main/java/com/cloud/network/nicira/NiciraRestClient.java  | 4 ++--
 .../src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java  | 4 ++--
 utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java   | 3 ++-
 3 files changed, 6 insertions(+), 5 deletions(-)

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1794387657

   @DaanHoogland a [SL] Trillian-Jenkins matrix job (centos7 mgmt + 
xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been 
kicked to run smoke tests


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7344:
URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1794387887

   @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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7344:
URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1794390928

   @DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Fix: Select another pod if all hosts in the pod becomes unavailable [cloudstack]

[via GitHub]

DaanHoogland commented on PR #8085:
URL: https://github.com/apache/cloudstack/pull/8085#issuecomment-1794415770

   > @blueorangutan test alma9 kvm-alma9
   
   this didn´t work :exploding_head: , so started it manually


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Fix: Select another pod if all hosts in the pod becomes unavailable [cloudstack]

[via GitHub]

blueorangutan commented on PR #8085:
URL: https://github.com/apache/cloudstack/pull/8085#issuecomment-1794416917

   @DaanHoogland [SL] unsupported parameters provided. Supported mgmt server os 
are: `centos7, centos6, suse15, alma8, ubuntu18, ubuntu22, ubuntu20, rocky8, 
alma9`. Supported hypervisors are: `kvm-centos6, kvm-centos7, kvm-rocky8, 
kvm-alma8, kvm-alma9, kvm-ubuntu18, kvm-ubuntu20, kvm-ubuntu22, kvm-suse15, 
vmware-55u3, vmware-60u2, vmware-65u2, vmware-67u3, vmware-70u1, vmware-70u2, 
vmware-70u3, vmware-80, vmware-80u1, xenserver-65sp1, xenserver-71, 
xenserver-74, xcpng74, xcpng76, xcpng80, xcpng81, xcpng82`


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Move subdomains between domains [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7446:
URL: https://github.com/apache/cloudstack/pull/7446#issuecomment-1794473237

   @JoaoJandre will you address @harikrishna-patnala 's comments?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

kiranchavala commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1794503753

   @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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1794505756

   @kiranchavala a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7344:
URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1794506367

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7644


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [NSX] Fix update source NAT IP [cloudstack]

[via GitHub]

nvazquez merged PR #8176:
URL: https://github.com/apache/cloudstack/pull/8176


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

(cloudstack) branch nsx-integration updated (42300aa80ed -> ecd32d5632a)

[nvazquez]

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

nvazquez pushed a change to branch nsx-integration
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


from 42300aa80ed Merge branch 'main' into nsx-integration
 add ecd32d5632a [NSX] Fix update source NAT IP (#8176)

No new revisions were added by this update.

Summary of changes:
 .../com/cloud/network/{ => nsx}/NsxProvider.java   |  2 +-
 .../java/com/cloud/network/nsx}/NsxService.java|  6 -
 .../com/cloud/network/element/NsxProviderVO.java   |  2 +-
 ...a => CreateOrUpdateNsxTier1NatRuleCommand.java} |  6 ++---
 .../api/command/AddNsxControllerCmd.java   |  2 +-
 .../api/response/NsxControllerResponse.java|  2 +-
 .../apache/cloudstack/resource/NsxResource.java|  8 +++
 .../cloudstack/service/NsxProviderService.java |  2 +-
 .../cloudstack/service/NsxProviderServiceImpl.java |  2 +-
 .../cloudstack/service/NsxPublicNetworkGuru.java   |  5 ++--
 .../apache/cloudstack/service/NsxServiceImpl.java  | 27 ++
 .../org/apache/cloudstack/utils/NsxHelper.java | 10 
 .../core/spring-nsx-core-managers-context.xml  |  8 +++
 .../META-INF/cloudstack/nsx/spring-nsx-context.xml |  2 --
 .../service/NsxProviderServiceImplTest.java|  2 +-
 .../java/com/cloud/network/vpc/VpcManagerImpl.java | 13 +++
 16 files changed, 70 insertions(+), 29 deletions(-)
 rename api/src/main/java/com/cloud/network/{ => nsx}/NsxProvider.java (97%)
 rename 
{plugins/network-elements/nsx/src/main/java/org/apache/cloudstack/service => 
api/src/main/java/com/cloud/network/nsx}/NsxService.java (85%)
 rename 
plugins/network-elements/nsx/src/main/java/org/apache/cloudstack/agent/api/{CreateNsxTier1NatRuleCommand.java
 => CreateOrUpdateNsxTier1NatRuleCommand.java} (83%)
 copy 
engine/storage/datamotion/src/main/resources/META-INF/cloudstack/core/spring-engine-storage-datamotion-core-context.xml
 => 
plugins/network-elements/nsx/src/main/resources/META-INF/cloudstack/core/spring-nsx-core-managers-context.xml
 (88%)

(cloudstack) branch nsx-integration updated (ecd32d5632a -> ab994dbfcfb)

[nvazquez]

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

nvazquez pushed a change to branch nsx-integration
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


from ecd32d5632a [NSX] Fix update source NAT IP (#8176)
 add 0caf18bc1a0 ui: indent issue in list view (#8184)
 add 2bb182c3e1e KVM Host HA enhancement for StorPool storage (#8045)
 add e9b24b6c326 Make authentication request parameter order to be 
deterministic (#8185)
 add ab994dbfcfb Merge branch 'main' into nsx-integration

No new revisions were added by this update.

Summary of changes:
 .../main/java/com/cloud/agent/api/to/HostTO.java   |  12 ++
 .../com/cloud/agent/api/CheckOnHostCommand.java|  11 ++
 .../java/com/cloud/agent/api/FenceCommand.java |  16 +++
 .../api/storage/PrimaryDataStoreDriver.java|   6 +
 .../java/com/cloud/ha/HighAvailabilityManager.java |   3 +
 .../main/java/com/cloud/ha/KVMInvestigator.java|  46 ---
 .../cloud/hypervisor/kvm/resource/KVMHABase.java   |  85 ++---
 .../hypervisor/kvm/resource/KVMHAChecker.java  |  42 ++-
 .../hypervisor/kvm/resource/KVMHAMonitor.java  | 123 +--
 .../kvm/resource/KVMHAVMActivityChecker.java   |  45 ++-
 .../wrapper/LibvirtCheckOnHostCommandWrapper.java  |   8 +-
 ...CheckVMActivityOnStoragePoolCommandWrapper.java |  15 ++-
 .../wrapper/LibvirtFenceCommandWrapper.java|   6 +-
 .../kvm/storage/IscsiAdmStoragePool.java   |  34 ++
 .../hypervisor/kvm/storage/KVMStoragePool.java |  23 
 .../kvm/storage/KVMStoragePoolManager.java |   6 +-
 .../kvm/storage/LibvirtStorageAdaptor.java |   1 +
 .../hypervisor/kvm/storage/LibvirtStoragePool.java |  95 ++
 .../hypervisor/kvm/storage/ScaleIOStoragePool.java |  34 ++
 .../apache/cloudstack/kvm/ha/KVMHAProvider.java|   5 +-
 .../cloudstack/kvm/ha/KVMHostActivityChecker.java  |   3 +-
 .../kvm/resource/LibvirtComputingResourceTest.java |   6 +-
 .../com/cloud/network/nicira/NiciraRestClient.java |   4 +-
 .../cloud/network/nicira/NiciraRestClientTest.java |   4 +-
 .../driver/DateraPrimaryDataStoreDriver.java   |  11 ++
 .../CloudStackPrimaryDataStoreDriverImpl.java  |   9 ++
 .../hypervisor/kvm/storage/LinstorStoragePool.java |  34 ++
 .../driver/LinstorPrimaryDataStoreDriverImpl.java  |  11 ++
 .../driver/NexentaPrimaryDataStoreDriver.java  |  11 ++
 .../driver/SamplePrimaryDataStoreDriverImpl.java   |  11 ++
 .../driver/ScaleIOPrimaryDataStoreDriver.java  |  10 ++
 .../driver/SolidFirePrimaryDataStoreDriver.java|   9 ++
 .../storage/StorPoolModifyStoragePoolAnswer.java   |  12 +-
 .../StorPoolModifyStorageCommandWrapper.java   |  39 ++
 .../kvm/storage/StorPoolStoragePool.java   | 135 
 .../driver/StorPoolPrimaryDataStoreDriver.java |  17 +++
 .../datastore/provider/StorPoolHostListener.java   |  23 +++-
 .../hypervisor/kvm/kvmspheartbeat.sh}  |  61 ++---
 .../com/cloud/ha/HighAvailabilityManagerImpl.java  |  30 -
 server/src/main/java/com/cloud/ha/KVMFencer.java   |   1 +
 .../cloud/ha/HighAvailabilityManagerImplTest.java  |  56 +
 ui/src/components/view/ListView.vue| 136 ++---
 .../cloud/utils/rest/HttpUriRequestBuilder.java|   3 +-
 43 files changed, 915 insertions(+), 337 deletions(-)
 copy scripts/{network/domr/router_proxy.sh => 
vm/hypervisor/kvm/kvmspheartbeat.sh} (56%)

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1794594054

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7645


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX integration [cloudstack]

[via GitHub]

nvazquez commented on PR #7919:
URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1794601790

   @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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX integration [cloudstack]

[via GitHub]

blueorangutan commented on PR #7919:
URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1794602653

   @nvazquez a [SL] Jenkins job has been kicked to build packages. It will be 
bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX integration [cloudstack]

[via GitHub]

sonarcloud[bot] commented on PR #7919:
URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1794677437

   SonarCloud Quality Gate failed.    [![Quality Gate 
failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png
 'Quality Gate 
failed')](https://sonarcloud.io/dashboard?id=apache_cloudstack&pullRequest=7919)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=BUG)
 
[![E](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/E-16px.png
 
'E')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=BUG)
 [8 
Bugs](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7919&resolved=false&types=SECURITY_HOTSPOT)
 
[![E](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/E-16px.png
 
'E')](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7919&resolved=false&types=SECURITY_HOTSPOT)
 [3 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_cloudstack&pullRequest=7919&resolved=false&types=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=CODE_SMELL)
 [151 Code 
Smells](https://sonarcloud.io/project/issues?id=apache_cloudstack&pullRequest=7919&resolved=false&types=CODE_SMELL)
   
   
[![15.0%](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/0-16px.png
 
'15.0%')](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7919&metric=new_coverage&view=list)
 [15.0% 
Coverage](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7919&metric=new_coverage&view=list)
  
   
[![4.3%](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/5-16px.png
 
'4.3%')](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7919&metric=new_duplicated_lines_density&view=list)
 [4.3% 
Duplication](https://sonarcloud.io/component_measures?id=apache_cloudstack&pullRequest=7919&metric=new_duplicated_lines_density&view=list)
   
   
![warning](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/message_warning-16px.png
 'warning') The version of Java (11.0.21) you have used to run this analysis is 
deprecated and we will stop accepting it soon. Please update to at least Java 
17.
   Read more [here](https://docs.sonarcloud.io/appendices/scanner-environment/)
   
   
   
   
   
![idea](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/light_bulb-16px.png
 'idea') Catch issues before they fail your Quality Gate with our IDE extension 
![sonarlint](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/sonarlint-16px.png
 'sonarlint') 
[SonarLint](https://www.sonarsource.com/products/sonarlint/features/connected-mode/?referrer=sonarcloud-welcome)


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX integration [cloudstack]

[via GitHub]

blueorangutan commented on PR #7919:
URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1794704856

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7646


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX integration [cloudstack]

[via GitHub]

blueorangutan commented on PR #7919:
URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1794719355

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7647


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[I] Change offerings (compute / Network / Disk) [cloudstack]

[via GitHub]

pcfriek1987 opened a new issue, #8188:
URL: https://github.com/apache/cloudstack/issues/8188

   
   
   # ISSUE TYPE
   
 * Feature Idea
   
   
   # COMPONENT NAME
   
   ~~~
   Service Offerings
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   4.18.1
   ~~~
   
   # CONFIGURATION
   
   N/A
   
   # OS / ENVIRONMENT
   
   N/A
   
   # SUMMARY
   
   
   
   # STEPS TO REPRODUCE
   
   
   
   ~~~
   Try to edit for example the default egress rules under network offerings or 
adjust the storage type of the compute offerings from shared to local when 
using only local storage.
   ~~~
   
   
   
   # EXPECTED RESULTS
   
   
   ~~~
   Being able to change those values so I do not need to create a new offering 
every time something needs to be adjusted.
   ~~~
   
   # ACTUAL RESULTS
   
   
   
   ~~~
   There is no way to adjust anything besides the name, description.
   ~~~
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Change offerings (compute / Network / Disk) [cloudstack]

[via GitHub]

boring-cyborg[bot] commented on issue #8188:
URL: https://github.com/apache/cloudstack/issues/8188#issuecomment-1794723918

   Thanks for opening your first issue here! Be sure to follow the issue 
template!
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1794766395

   I test upgraded from 4.16 and 4.17, @weizhouapache . Can we be satisfied 
with that?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Being able to change offerings (compute / Network / Disk) [cloudstack]

[via GitHub]

DaanHoogland commented on issue #8188:
URL: https://github.com/apache/cloudstack/issues/8188#issuecomment-1794807448

   We already discussed some possible definitions of this enhancement on the 
us...@cloudstack.apache.org list. see 
https://lists.apache.org/thread/nj3t2xp200fcx254fdxqll66694hclol
   
   I want to add that a possible implementation could be a setting that lists 
the allowable fields for update; i.e. service.offering.allowed.update.fields = 
name,description,default_acl,...
   
   (names are just illustrative)


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] PEP 440 When Installing cloudstack-marvin breaks building wheels [cloudstack]

[via GitHub]

tampler commented on issue #7927:
URL: https://github.com/apache/cloudstack/issues/7927#issuecomment-1794868805

   Still experiencing this issue with `4.19.0.0-snapshot.20231106_xenial_all` 
on Ubuntu 22.04. Bumped `pip3` to `v23.3.1` and now getting an issue removing 
`marvin`:
   ```sh
   Do you want to continue? [Y/n] y
   Setting up cloudstack-marvin (4.19.0.0-snapshot.20231106~xenial) ...
   Requirement already satisfied: pip in 
/usr/local/lib/python3.10/dist-packages (23.3.1)
   WARNING: Running pip as the 'root' user can result in broken permissions and 
conflicting behaviour with the system package manager. It is recommended to use 
a virtual environment instead: https://pip.pypa.io/warnings/venv
   WARNING: Requirement '/usr/share/cloudstack-marvin/Marvin-*.tar.gz' looks 
like a filename, but the file does not exist
   Processing /usr/share/cloudstack-marvin/Marvin-*.tar.gz
   ERROR: Could not install packages due to an OSError: [Errno 2] No such file 
or directory: '/usr/share/cloudstack-marvin/Marvin-*.tar.gz'
   
   dpkg: error processing package cloudstack-marvin (--configure):
installed cloudstack-marvin package post-installation script subprocess 
returned error exit status 1
   Errors were encountered while processing:
cloudstack-marvin
   ```
   
   Looks like a version mismatch between apt-dpkg-pip3


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Linstor backup snaphots [cloudstack]

[via GitHub]

rp- commented on PR #8067:
URL: https://github.com/apache/cloudstack/pull/8067#issuecomment-1794945766

   if you can't provide packages, could you check if `java-linstor-0.3.0.jar` 
is included in the `cloudstack-agent/lib` ?
   or maybe it is possible to get more debug output, why it gets stuck on 
loading the LinstorStorageAdaptor?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

(cloudstack-cloudmonkey) branch bug/139-support-post-method updated (9f81008 -> 35ed628)

[dahn]

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

dahn pushed a change to branch bug/139-support-post-method
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git


from 9f81008  add support for POST method in CloudMonkey
 add 35ed628  extract request execution

No new revisions were added by this update.

Summary of changes:
 cmd/network.go | 38 --
 1 file changed, 16 insertions(+), 22 deletions(-)

(cloudstack-cloudmonkey) branch main updated (2d641ae -> cbb51b0)

[dahn]

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

dahn pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git


from 2d641ae  about: bump version to next v6.4.0-(rc)
 add e544cc9  Optimizing network.go
 new cbb51b0  Optimizing network.go #129 from davidjumani/optimize-nw

The 1 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:
 cmd/network.go | 52 +---
 1 file changed, 29 insertions(+), 23 deletions(-)

(cloudstack-cloudmonkey) 01/01: Optimizing network.go #129 from davidjumani/optimize-nw

[dahn]

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

dahn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git

commit cbb51b0f506758564b9afd7bd08ea8756d127ff6
Merge: 2d641ae e544cc9
Author: dahn 
AuthorDate: Mon Nov 6 16:32:52 2023 +0100

Optimizing network.go #129 from davidjumani/optimize-nw

 cmd/network.go | 52 +---
 1 file changed, 29 insertions(+), 23 deletions(-)

(cloudstack-cloudmonkey) branch bug/139-support-post-method deleted (was 35ed628)

[dahn]

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

dahn pushed a change to branch bug/139-support-post-method
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git


 was 35ed628  extract request execution

The revisions that were on this branch are still contained in
other references; therefore, this change does not discard any commits
from the repository.

(cloudstack-cloudmonkey) branch main updated (cbb51b0 -> e19c4f6)

[dahn]

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

dahn pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git


from cbb51b0  Optimizing network.go #129 from davidjumani/optimize-nw
 add 9f81008  add support for POST method in CloudMonkey
 add 35ed628  extract request execution
 new e19c4f6  Convert GET to POST requests #140

The 1 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:
 cmd/network.go | 22 +-
 1 file changed, 17 insertions(+), 5 deletions(-)

(cloudstack-cloudmonkey) 01/01: Convert GET to POST requests #140

[dahn]

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

dahn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git

commit e19c4f6845d98c739b965aa5860a3499f8123955
Merge: cbb51b0 35ed628
Author: dahn 
AuthorDate: Mon Nov 6 16:38:30 2023 +0100

Convert GET to POST requests #140

 cmd/network.go | 22 +-
 1 file changed, 17 insertions(+), 5 deletions(-)

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

weizhouapache commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1795314603

   > I test upgraded from 4.16 and 4.17, @weizhouapache . Can we be satisfied 
with that?
   
   @DaanHoogland 
   cool.
   The changes have only impact on the views which are changed in the upgrade 
from 4.18.1.0 to 4.19.0.0
   the impact is small I think. looks ok to me.
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

kiranchavala commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1795336344

   @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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1795339786

   @kiranchavala a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Unable to migrate VMWare VM from cloudstack but able to create new VMs [cloudstack]

[via GitHub]

yashi4engg closed issue #8105: Unable to migrate VMWare VM from cloudstack but 
able to create new VMs 
URL: https://github.com/apache/cloudstack/issues/8105


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Unable to migrate VMWare VM from cloudstack but able to create new VMs [cloudstack]

[via GitHub]

yashi4engg commented on issue #8105:
URL: https://github.com/apache/cloudstack/issues/8105#issuecomment-1795398090

   Hi Team , 
   
   Anyone look into this and suggest any solution. 


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Move subdomains between domains [cloudstack]

[via GitHub]

JoaoJandre commented on PR #7446:
URL: https://github.com/apache/cloudstack/pull/7446#issuecomment-1795484866

   @DaanHoogland sure, I've been a bit busy lately, but I will address the 
comments as soon as I can  :)


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1795581404

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7650


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] Database VIEW management via separated files [cloudstack]

[via GitHub]

DaanHoogland commented on PR #7417:
URL: https://github.com/apache/cloudstack/pull/7417#issuecomment-1796157201

   > This PR have many database changes, which require intensive upgrade tests 
(from various older versions) IMHO.
   
   ok, I'll do a simple upgrade test before merging. We'll have to test the RC 
intensively  for upgrade anyway.
   
   > > I test upgraded from 4.16 and 4.17, @weizhouapache . Can we be satisfied 
with that?
   > 
   > @DaanHoogland cool. The changes have only impact on the views which are 
changed in the upgrade from 4.18.1.0 to 4.19.0.0 the impact is small I think. 
looks ok to me.
   
   @shwstppr , can you concur?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1796386898

   @kiranchavala a [SL] Jenkins job has been kicked to build packages. It will 
be bundled with  KVM, XenServer and VMware SystemVM templates. 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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

kiranchavala commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1796386318

   @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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[PR] NSX: CKS Support [cloudstack]

[via GitHub]

Pearl1594 opened a new pull request, #8189:
URL: https://github.com/apache/cloudstack/pull/8189

   ### Description
   
   This PR...
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] 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)
   - [ ] build/CI
   
   ### Feature/Enhancement Scale or Bug Severity
   
    Feature/Enhancement Scale
   
   - [ ] Major
   - [ ] Minor
   
    Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [ ] Trivial
   
   
   ### Screenshots (if appropriate):
   
   
   ### How Has This Been Tested?
   
   
   
   
    How did you try to break this feature and the system with this change?
   
   
   
   
   
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] NSX: CKS Support [cloudstack]

[via GitHub]

codecov[bot] commented on PR #8189:
URL: https://github.com/apache/cloudstack/pull/8189#issuecomment-1796423436

   ## 
[Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   > Merging 
[#8189](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 (24495c1) into 
[nsx-integration](https://app.codecov.io/gh/apache/cloudstack/commit/e9b24b6c3266f14a85f2db86fb5d824539206097?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 (e9b24b6) will **decrease** coverage by `23.86%`.
   > Report is 86 commits behind head on nsx-integration.
   > The diff coverage is `0.00%`.
   
   ```diff
   @@  Coverage Diff  @@
   ## nsx-integration   #8189   +/-   ##
   =
   - Coverage  28.33%   4.48%   -23.86% 
   =
 Files   5181 354 -4827 
 Lines 365434   27981   -337453 
 Branches   534494904-48545 
   =
   - Hits  1035501255   -102295 
   + Misses247552   26588   -220964 
   + Partials   14332 138-14194 
   ```
   
   | 
[Flag](https://app.codecov.io/gh/apache/cloudstack/pull/8189/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Coverage Δ | |
   |---|---|---|
   | 
[simulator-marvin-tests](https://app.codecov.io/gh/apache/cloudstack/pull/8189/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `?` | |
   | 
[uitests](https://app.codecov.io/gh/apache/cloudstack/pull/8189/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `4.48% <0.00%> (-0.02%)` | :arrow_down: |
   | 
[unit-tests](https://app.codecov.io/gh/apache/cloudstack/pull/8189/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `?` | |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | 
[Files](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Coverage Δ | |
   |---|---|---|
   | 
[ui/src/components/view/DetailsTab.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL2NvbXBvbmVudHMvdmlldy9EZXRhaWxzVGFiLnZ1ZQ==)
 | `11.88% <ø> (ø)` | |
   | 
[ui/src/config/section/offering.js](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL2NvbmZpZy9zZWN0aW9uL29mZmVyaW5nLmpz)
 | `0.00% <ø> (ø)` | |
   | 
[.../infra/zone/ZoneWizardPhysicalNetworkSetupStep.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL3ZpZXdzL2luZnJhL3pvbmUvWm9uZVdpemFyZFBoeXNpY2FsTmV0d29ya1NldHVwU3RlcC52dWU=)
 | `0.00% <ø> (ø)` | |
   | 
[ui/src/views/infra/zone/IpAddressRangeForm.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL3ZpZXdzL2luZnJhL3pvbmUvSXBBZGRyZXNzUmFuZ2VGb3JtLnZ1ZQ==)
 | `0.00% <0.00%> (ø)` | |
   | 
[ui/src/views/network/CreateIsolatedNetworkForm.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL3ZpZXdzL25ldHdvcmsvQ3JlYXRlSXNvbGF0ZWROZXR3b3JrRm9ybS52dWU=)
 | `0.00% <0.00%> (ø)` | |
   | 
[ui/src/components/CheckBoxSelectPair.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-dWkvc3JjL2NvbXBvbmVudHMvQ2hlY2tCb3hTZWxlY3RQYWlyLnZ1ZQ==)
 | `0.00% <0.00%> (ø)` | |
   | 
[ui/src/views/infra/network/ServiceProvidersTab.vue](https://app.codecov.io/gh/apache/cloudstack/pull/8189?src=pr&el=tree&utm_medium=

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1796512577

   [SF] Trillian test result (tid-8253)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 41871 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7345-t8253-kvm-centos7.zip
   Smoke tests completed. 109 look OK, 0 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [Draft] KVM Ingestion - Import Instance [cloudstack]

[via GitHub]

blueorangutan commented on PR #7976:
URL: https://github.com/apache/cloudstack/pull/7976#issuecomment-1796522106

   Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 
:heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. 
SL-JID 7651


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [PR] [HEALTH] 4.18 LTS Health Check, please don't merge this! [cloudstack]

[via GitHub]

blueorangutan commented on PR #7345:
URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1796915670

   [SF] Trillian test result (tid-8252)
   Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
   Total time taken: 45854 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7345-t8252-vmware-67u3.zip
   Smoke tests completed. 102 look OK, 7 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_03_ping_in_ssvm_success | `Error` | 1.13 | test_diagnostics.py
   test_04_ping_in_ssvm_failure | `Error` | 1.15 | test_diagnostics.py
   test_08_arping_in_ssvm | `Error` | 1.14 | test_diagnostics.py
   test_09_arping_in_cpvm | `Error` | 1.12 | test_diagnostics.py
   test_11_traceroute_in_ssvm | `Error` | 1.14 | test_diagnostics.py
   test_15_retrieve_ssvm_default_files | `Error` | 1.13 | test_diagnostics.py
   test_16_retrieve_ssvm_single_file | `Error` | 1.13 | test_diagnostics.py
   test_01_add_primary_storage_disabled_host | `Error` | 35.93 | 
test_primary_storage.py
   test_01_non_strict_host_anti_affinity | `Failure` | 146.31 | 
test_nonstrict_affinity_group.py
   test_02_non_strict_host_affinity | `Error` | 100.01 | 
test_nonstrict_affinity_group.py
   test_03_deploy_and_scale_kubernetes_cluster | `Failure` | 40.20 | 
test_kubernetes_clusters.py
   test_07_deploy_kubernetes_ha_cluster | `Failure` | 97.73 | 
test_kubernetes_clusters.py
   test_08_upgrade_kubernetes_ha_cluster | `Failure` | 59.64 | 
test_kubernetes_clusters.py
   test_09_delete_kubernetes_ha_cluster | `Failure` | 56.55 | 
test_kubernetes_clusters.py
   test_10_vpc_tier_kubernetes_cluster | `Error` | 2.49 | 
test_kubernetes_clusters.py
   ContextSuite context=TestKubernetesCluster>:teardown | `Error` | 85.94 | 
test_kubernetes_clusters.py
   test_01_sys_vm_start | `Failure` | 0.09 | test_secondary_storage.py
   test_01_deploy_vm_on_specific_host | `Error` | 0.10 | 
test_vm_deployment_planner.py
   test_04_deploy_vm_on_host_override_pod_and_cluster | `Error` | 0.12 | 
test_vm_deployment_planner.py
   test_03_live_migrate_VM_with_two_data_disks | `Error` | 62.92 | 
test_vm_life_cycle.py
   test_08_migrate_vm | `Error` | 0.06 | test_vm_life_cycle.py
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

rongage commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1797009065

   That is the first thing I tried when this started messing up (I was able 
   to upload 3 ISOs before it stopped working).
   
   Apparently Chrome has memorized the cert for the SSVM because browsing 
   to the IP address - in my case: https://192.168.0.152 - does not present 
   an "accept this cert" dialog.  In fact, if this was the problem, then 
   wouldn't the upload fail to even start (I get the upload progress dialog 
   without a problem).
   
   I had found similar issues to this one on GitHub (#7442, #7981). The 
   resolution to that issue mentioned something about a firewall rule to 
   the SSVM.  For grins, I tried that and no difference was encountered (or 
   I misunderstood where to modify the firewall rule).
   
   Any chance for other things to try?
   
   Ron Gage
   
   
   On 11/6/2023 3:20 AM, dahn wrote:
   >
   > Ah, this has to do with certificates. Uploading from local will fail 
   > this way. The trick is to try an
   >
   >   * upload while having the debugger/network log open in your browser
   >   * get the actual upload url
   >   * open another window
   >   * go to the base url of the upload
   >   * accept the server cert
   >   * go back and try to upload
   >
   > let me know if this works @rongage 
   >
   > —
   > Reply to this email directly, view it on GitHub 
   > 
, 
   > or unsubscribe 
   > 
.
   > You are receiving this because you were mentioned.Message ID: 
   > ***@***.***>
   >


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

weizhouapache commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1797012554

   > That is the first thing I tried when this started messing up (I was able 
   > to upload 3 ISOs before it stopped working).
   > 
   > Apparently Chrome has memorized the cert for the SSVM because browsing 
   > to the IP address - in my case: https://192.168.0.152 - does not present 
   > an "accept this cert" dialog.  In fact, if this was the problem, then 
   > wouldn't the upload fail to even start (I get the upload progress dialog 
   > without a problem).
   > 
   > I had found similar issues to this one on GitHub (#7442, #7981). The 
   > resolution to that issue mentioned something about a firewall rule to 
   > the SSVM.  For grins, I tried that and no difference was encountered (or 
   > I misunderstood where to modify the firewall rule).
   > 
   > Any chance for other things to try?
   > 
   > Ron Gage
   > 
   > 
   > On 11/6/2023 3:20 AM, dahn wrote:
   > >
   > > Ah, this has to do with certificates. Uploading from local will fail 
   > > this way. The trick is to try an
   > >
   > >   * upload while having the debugger/network log open in your browser
   > >   * get the actual upload url
   > >   * open another window
   > >   * go to the base url of the upload
   > >   * accept the server cert
   > >   * go back and try to upload
   > >
   > > let me know if this works @rongage 
   > >
   > > —
   > > Reply to this email directly, view it on GitHub 
   > > 
, 
   > > or unsubscribe 
   > > 
.
   > > You are receiving this because you were mentioned.Message ID: 
   > > ***@***.***>
   > >
   
   Can you check if there are errors in /var/log/cloud.log in SSVM? 


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

rongage commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1797025269

   Just tried uploading an ISO again.  This exception throws immediately at 
   the start of the upload:
   
   2023-11-06 23:25:48,087 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) HEADER: SIGNATURE=ufQC9ZETkswaVrCh78Bps4T7kfw=
   2023-11-06 23:25:48,088 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) HEADER: 
   
METADATA=Kks2KPhSwMltHKEpFTOvS8rR1gcdyiG6eqciFzztBm1YUip/FEkycNf0X8BXEDJYBIg8DrILrhlxuT+KPn8RejFOjyVn5TTrgYVyhhC0j8Wk6NZ/m70VbUjJ6EUj+iRnjrrutKBxs+E6a+l4qhL7kOavy6yvG1+c426wByjQIVUcyCRbpnnzygIh/nZrxsynH5jjxibhfqTjDqjNLPUV4eorbPAtmEr20gGn8RGtf7+rdaX4cJPUKDaKYoESUcbKezbtreDfcl9BPJuyrZ9rd/6KaG5YCrInaf58yMxMDtZOZMzZ0HB4Ln5nIE/LwBhQIaHZ/o+LzC/j1FXPYkDwbEzENYzosCUTXJBMWcLn3v16hjhUJockoE8rc1N/A5P8KdBronQ/PGwgSW8e0d1vDbRvVwkzduagCadcRTbub1GBVhCKphak/2prUt6Z3Kn5sJNoGCvT0NManLp0QV2j/5DtB+NSQZLnn+CEf6johZ4B044BqRlWAV7CDPOR8ME9BhPjDCfdCkr7V+lvMYnUgsyxdihLsgeYU4WZk6xYfLfJAjo0FJQbJusaf7jsz2Yh6jQPtfd+F3Lni+1GWc9MQH3czJ8qtISkse0=
   2023-11-06 23:25:48,089 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) HEADER: EXPIRES=2023-11-06T23:40:47.341Z
   2023-11-06 23:25:48,090 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) HEADER: HOST=192.168.0.152
   2023-11-06 23:25:48,090 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) HEADER: CONTENT_LENGTH=537211
   2023-11-06 23:25:48,092 INFO [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) URI: uuid=67f72e90-a981-4d76-a7ea-03a619d1172d
   2023-11-06 23:25:48,147 ERROR [storage.resource.HttpUploadServerHandler] 
   (nioEventLoopGroup-3-1:null) post request validation failed
   com.cloud.exception.InvalidParameterValueException: signature validation 
   failed.
    at 
   
org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.validatePostUploadRequest(NfsSecondaryStorageResource.java:3536)
    at 
   
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:165)
    at 
   
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:65)
    at 
   
io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
    at 
   
io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:102)
    at 
   
io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
    at 
   
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:328)
    at 
   
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:315)
    at 
   
io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:429)
    at 
   
io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:283)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
    at 
   
io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1422)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
    at 
   
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
    at 
   
io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:931)
    at 
   
io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:163)
    at 
   io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:700)
    at 
   
io.netty.channel.nio.NioEventLoop.processSelectedKeysOptim

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

rongage commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1797059652

   I did just now try with Edge instead of Chrome - same result.
   
   I also tried re-enabling https on the upload setting 
   (use.https.to.upload) - That only prevents the progress dialog from 
   coming up.
   
   I also did the network call trace for the upload.  The erroring URL is: 
   https://192.168.0.152/upload/5e13667c-a2e0-4329-9752-0529676a88b4 Trying 
   to open that URL does do anything.  The only thing that comes out of 
   that is that the error point in the UI code is xhr.js:187.  No response 
   was received from that URL call.
   
   Is it at all possible that this URL should be http instead of https 
   since I have the https setting off.
   
   Ron Gage
   
   
   On 11/6/2023 6:28 PM, Ron Gage wrote:
   >
   > Just tried uploading an ISO again.  This exception throws immediately 
   > at the start of the upload:
   >
   > 2023-11-06 23:25:48,087 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) HEADER: 
   > SIGNATURE=ufQC9ZETkswaVrCh78Bps4T7kfw=
   > 2023-11-06 23:25:48,088 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) HEADER: 
   > 
METADATA=Kks2KPhSwMltHKEpFTOvS8rR1gcdyiG6eqciFzztBm1YUip/FEkycNf0X8BXEDJYBIg8DrILrhlxuT+KPn8RejFOjyVn5TTrgYVyhhC0j8Wk6NZ/m70VbUjJ6EUj+iRnjrrutKBxs+E6a+l4qhL7kOavy6yvG1+c426wByjQIVUcyCRbpnnzygIh/nZrxsynH5jjxibhfqTjDqjNLPUV4eorbPAtmEr20gGn8RGtf7+rdaX4cJPUKDaKYoESUcbKezbtreDfcl9BPJuyrZ9rd/6KaG5YCrInaf58yMxMDtZOZMzZ0HB4Ln5nIE/LwBhQIaHZ/o+LzC/j1FXPYkDwbEzENYzosCUTXJBMWcLn3v16hjhUJockoE8rc1N/A5P8KdBronQ/PGwgSW8e0d1vDbRvVwkzduagCadcRTbub1GBVhCKphak/2prUt6Z3Kn5sJNoGCvT0NManLp0QV2j/5DtB+NSQZLnn+CEf6johZ4B044BqRlWAV7CDPOR8ME9BhPjDCfdCkr7V+lvMYnUgsyxdihLsgeYU4WZk6xYfLfJAjo0FJQbJusaf7jsz2Yh6jQPtfd+F3Lni+1GWc9MQH3czJ8qtISkse0=
   > 2023-11-06 23:25:48,089 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) HEADER: EXPIRES=2023-11-06T23:40:47.341Z
   > 2023-11-06 23:25:48,090 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) HEADER: HOST=192.168.0.152
   > 2023-11-06 23:25:48,090 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) HEADER: CONTENT_LENGTH=537211
   > 2023-11-06 23:25:48,092 INFO 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) URI: 
   > uuid=67f72e90-a981-4d76-a7ea-03a619d1172d
   > 2023-11-06 23:25:48,147 ERROR 
   > [storage.resource.HttpUploadServerHandler] 
   > (nioEventLoopGroup-3-1:null) post request validation failed
   > com.cloud.exception.InvalidParameterValueException: signature 
   > validation failed.
   >     at 
   > 
org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.validatePostUploadRequest(NfsSecondaryStorageResource.java:3536)
   >     at 
   > 
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:165)
   >     at 
   > 
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:65)
   >     at 
   > 
io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
   >     at 
   > 
io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:102)
   >     at 
   > 
io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
   >     at 
   > 
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:328)
   >     at 
   > 
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:315)
   >     at 
   > 
io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:429)
   >     at 
   > 
io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:283)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
   >     at 
   > 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
   

Re: [I] Getting Error 502 at the end of ISO upload under the Images menu [cloudstack]

[via GitHub]

rongage commented on issue #8175:
URL: https://github.com/apache/cloudstack/issues/8175#issuecomment-1797064271

   Strike that last line about https.  I don't know what I am doing 
   sometimes...
   
   Ron
   
   
   On 11/6/2023 7:06 PM, Ron Gage wrote:
   >
   > I did just now try with Edge instead of Chrome - same result.
   >
   > I also tried re-enabling https on the upload setting 
   > (use.https.to.upload) - That only prevents the progress dialog from 
   > coming up.
   >
   > I also did the network call trace for the upload.  The erroring URL 
   > is: https://192.168.0.152/upload/5e13667c-a2e0-4329-9752-0529676a88b4 
   > Trying to open that URL does do anything.  The only thing that comes 
   > out of that is that the error point in the UI code is xhr.js:187.  No 
   > response was received from that URL call.
   >
   > Is it at all possible that this URL should be http instead of https 
   > since I have the https setting off.
   >
   > Ron Gage
   >
   >
   > On 11/6/2023 6:28 PM, Ron Gage wrote:
   >>
   >> Just tried uploading an ISO again.  This exception throws immediately 
   >> at the start of the upload:
   >>
   >> 2023-11-06 23:25:48,087 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) HEADER: 
   >> SIGNATURE=ufQC9ZETkswaVrCh78Bps4T7kfw=
   >> 2023-11-06 23:25:48,088 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) HEADER: 
   >> 
METADATA=Kks2KPhSwMltHKEpFTOvS8rR1gcdyiG6eqciFzztBm1YUip/FEkycNf0X8BXEDJYBIg8DrILrhlxuT+KPn8RejFOjyVn5TTrgYVyhhC0j8Wk6NZ/m70VbUjJ6EUj+iRnjrrutKBxs+E6a+l4qhL7kOavy6yvG1+c426wByjQIVUcyCRbpnnzygIh/nZrxsynH5jjxibhfqTjDqjNLPUV4eorbPAtmEr20gGn8RGtf7+rdaX4cJPUKDaKYoESUcbKezbtreDfcl9BPJuyrZ9rd/6KaG5YCrInaf58yMxMDtZOZMzZ0HB4Ln5nIE/LwBhQIaHZ/o+LzC/j1FXPYkDwbEzENYzosCUTXJBMWcLn3v16hjhUJockoE8rc1N/A5P8KdBronQ/PGwgSW8e0d1vDbRvVwkzduagCadcRTbub1GBVhCKphak/2prUt6Z3Kn5sJNoGCvT0NManLp0QV2j/5DtB+NSQZLnn+CEf6johZ4B044BqRlWAV7CDPOR8ME9BhPjDCfdCkr7V+lvMYnUgsyxdihLsgeYU4WZk6xYfLfJAjo0FJQbJusaf7jsz2Yh6jQPtfd+F3Lni+1GWc9MQH3czJ8qtISkse0=
   >> 2023-11-06 23:25:48,089 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) HEADER: EXPIRES=2023-11-06T23:40:47.341Z
   >> 2023-11-06 23:25:48,090 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) HEADER: HOST=192.168.0.152
   >> 2023-11-06 23:25:48,090 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) HEADER: CONTENT_LENGTH=537211
   >> 2023-11-06 23:25:48,092 INFO 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) URI: 
   >> uuid=67f72e90-a981-4d76-a7ea-03a619d1172d
   >> 2023-11-06 23:25:48,147 ERROR 
   >> [storage.resource.HttpUploadServerHandler] 
   >> (nioEventLoopGroup-3-1:null) post request validation failed
   >> com.cloud.exception.InvalidParameterValueException: signature 
   >> validation failed.
   >>     at 
   >> 
org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.validatePostUploadRequest(NfsSecondaryStorageResource.java:3536)
   >>     at 
   >> 
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:165)
   >>     at 
   >> 
org.apache.cloudstack.storage.resource.HttpUploadServerHandler.channelRead0(HttpUploadServerHandler.java:65)
   >>     at 
   >> 
io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
   >>     at 
   >> 
io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:102)
   >>     at 
   >> 
io.netty.handler.codec.MessageToMessageCodec.channelRead(MessageToMessageCodec.java:111)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:360)
   >>     at 
   >> 
io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:352)
   >>     at 
   >> 
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:328)
   >>     at 
   >> 
io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:315)
   >>     at 
   >> 
io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:429)
   >>     at 
   >> 
io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:283)
   >>     a

Re: [PR] Create global configuration to allow changing the default nic adapter for user VMs in VMware [cloudstack]

[via GitHub]

blueorangutan commented on PR #7954:
URL: https://github.com/apache/cloudstack/pull/7954#issuecomment-1797122066

   [SF] Trillian test result (tid-8250)
   Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server a8
   Total time taken: 59318 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7954-t8250-vmware-70u3.zip
   Smoke tests completed. 110 look OK, 7 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_browser_migrate_template | `Failure` | 5.44 | 
test_image_store_object_migration.py
   test_02_balanced_drs_algorithm | `Failure` | 138.22 | test_cluster_drs.py
   test_03_deploy_and_destroy_VM_and_verify_network_resources_persist | `Error` 
| 0.04 | test_persistent_network.py
   test_01_add_primary_storage_disabled_host | `Error` | 37.25 | 
test_primary_storage.py
   test_01_sys_vm_start | `Failure` | 0.11 | test_secondary_storage.py
   test_01_deploy_vm_on_specific_host | `Error` | 0.09 | 
test_vm_deployment_planner.py
   test_02_deploy_vm_on_specific_cluster | `Error` | 16.69 | 
test_vm_deployment_planner.py
   test_03_deploy_vm_on_specific_pod | `Error` | 3602.32 | 
test_vm_deployment_planner.py
   test_04_deploy_vm_on_host_override_pod_and_cluster | `Error` | 0.15 | 
test_vm_deployment_planner.py
   test_05_deploy_vm_on_cluster_override_pod | `Error` | 1.34 | 
test_vm_deployment_planner.py
   test_03_live_migrate_VM_with_two_data_disks | `Error` | 62.08 | 
test_vm_life_cycle.py
   test_08_migrate_vm | `Error` | 0.08 | test_vm_life_cycle.py
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org