[GitHub] [cloudstack] blueorangutan commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
blueorangutan commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1213598436 Trillian test result (tid-4678) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42816 seconds Marvin logs:

[GitHub] [cloudstack] sonarcloud[bot] commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
sonarcloud[bot] commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213542961 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [cloudstack] shwstppr commented on issue #6626: Compute offering "Custom constrained" not working

2022-08-12 Thread GitBox
shwstppr commented on issue #6626: URL: https://github.com/apache/cloudstack/issues/6626#issuecomment-1213527144 @cdfgallo can you try with a template or compute offering which dynamically scalable parameter set to false. I was able to reproduce this with dynamically scalable enabled for

[GitHub] [cloudstack] blueorangutan commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
blueorangutan commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213514981 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6348 (SL-JID-2138) -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
blueorangutan commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213511643 @acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] acs-robot commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
acs-robot commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213511388 Found UI changes, kicking a new UI QA build @blueorangutan ui -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] blueorangutan commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
blueorangutan commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213489318 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6348 (SL-JID-2137) -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
blueorangutan commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213488043 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6522 (SL-JID-2136) -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
blueorangutan commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213485590 @acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] acs-robot commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
acs-robot commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213485115 Found UI changes, kicking a new UI QA build @blueorangutan ui -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] GutoVeronezi commented on pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
GutoVeronezi commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1213484939 @DaanHoogland, reviews addressed, many thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [cloudstack] blueorangutan commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
blueorangutan commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213484155 @acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] acs-robot commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
acs-robot commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213483541 Found UI changes, kicking a new UI QA build @blueorangutan ui -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] blueorangutan commented on pull request #6511: [HEALTH] 4.18/main Health Check, please don't merge this!

2022-08-12 Thread GitBox
blueorangutan commented on PR #6511: URL: https://github.com/apache/cloudstack/pull/6511#issuecomment-1213473119 @shwstppr a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This is an automated

[GitHub] [cloudstack] shwstppr commented on pull request #6511: [HEALTH] 4.18/main Health Check, please don't merge this!

2022-08-12 Thread GitBox
shwstppr commented on PR #6511: URL: https://github.com/apache/cloudstack/pull/6511#issuecomment-1213472385 @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

[GitHub] [cloudstack] blueorangutan commented on pull request #6526: [HEALTH] 4.17 Health Check please don't merge this

2022-08-12 Thread GitBox
blueorangutan commented on PR #6526: URL: https://github.com/apache/cloudstack/pull/6526#issuecomment-1213472365 @shwstppr a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This is an automated

[GitHub] [cloudstack] shwstppr commented on pull request #6526: [HEALTH] 4.17 Health Check please don't merge this

2022-08-12 Thread GitBox
shwstppr commented on PR #6526: URL: https://github.com/apache/cloudstack/pull/6526#issuecomment-1213472077 @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

[GitHub] [cloudstack] boring-cyborg[bot] commented on pull request #6638: Fix: rpm dependencies: which and file

2022-08-12 Thread GitBox
boring-cyborg[bot] commented on PR #6638: URL: https://github.com/apache/cloudstack/pull/6638#issuecomment-1213443272 Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our

[GitHub] [cloudstack] annProg opened a new pull request, #6638: Fix: rpm dependencies: which and file

2022-08-12 Thread GitBox
annProg opened a new pull request, #6638: URL: https://github.com/apache/cloudstack/pull/6638 ### Description `createtmplt.sh` will use `file`, and `setup-sysvm-tmplt` will use `which`. this is log: ``` /usr/share/cloudstack-common/scripts/storage/secondary/setup-sysvm-tmplt:

[GitHub] [cloudstack] blueorangutan commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
blueorangutan commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213439978 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6522 (SL-JID-2135) -- This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack] blueorangutan commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
blueorangutan commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213435951 @acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] acs-robot commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
acs-robot commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1213435265 Found UI changes, kicking a new UI QA build @blueorangutan ui -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] blueorangutan commented on pull request #6526: [HEALTH] 4.17 Health Check please don't merge this

2022-08-12 Thread GitBox
blueorangutan commented on PR #6526: URL: https://github.com/apache/cloudstack/pull/6526#issuecomment-1213432029 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3972 -- This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on pull request #6511: [HEALTH] 4.18/main Health Check, please don't merge this!

2022-08-12 Thread GitBox
blueorangutan commented on PR #6511: URL: https://github.com/apache/cloudstack/pull/6511#issuecomment-1213430344 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3971 -- This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on pull request #6526: [HEALTH] 4.17 Health Check please don't merge this

2022-08-12 Thread GitBox
blueorangutan commented on PR #6526: URL: https://github.com/apache/cloudstack/pull/6526#issuecomment-1213385895 @shwstppr a 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. --

[GitHub] [cloudstack] blueorangutan commented on pull request #6511: [HEALTH] 4.18/main Health Check, please don't merge this!

2022-08-12 Thread GitBox
blueorangutan commented on PR #6511: URL: https://github.com/apache/cloudstack/pull/6511#issuecomment-1213385878 @shwstppr a 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. --

[GitHub] [cloudstack] shwstppr commented on pull request #6526: [HEALTH] 4.17 Health Check please don't merge this

2022-08-12 Thread GitBox
shwstppr commented on PR #6526: URL: https://github.com/apache/cloudstack/pull/6526#issuecomment-1213385393 @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

[GitHub] [cloudstack] shwstppr commented on pull request #6511: [HEALTH] 4.18/main Health Check, please don't merge this!

2022-08-12 Thread GitBox
shwstppr commented on PR #6511: URL: https://github.com/apache/cloudstack/pull/6511#issuecomment-1213385613 @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

[cloudstack] 01/01: Merge remote-tracking branch 'apache/4.17' into main

2022-08-12 Thread shwstppr
This is an automated email from the ASF dual-hosted git repository. shwstppr pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit d9b770eb486886fb21db19656e00062373abf4f9 Merge: 9717ed9af2 9410a70454 Author: Abhishek Kumar AuthorDate: Fri Aug 12

[cloudstack] branch main updated (9717ed9af2 -> d9b770eb48)

2022-08-12 Thread shwstppr
This is an automated email from the ASF dual-hosted git repository. shwstppr pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 9717ed9af2 Improve log messages on VolumeOrchestrator class (#6408) add 696b93f421 kvm: update host memory

[cloudstack] branch 4.17 updated: schema: increase size of column 'value' at table 'account_details' (#6080)

2022-08-12 Thread shwstppr
This is an automated email from the ASF dual-hosted git repository. shwstppr pushed a commit to branch 4.17 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.17 by this push: new 9410a70454 schema: increase size of column

[GitHub] [cloudstack] shwstppr merged pull request #6080: Increase size of column 'value' at table 'account_details'

2022-08-12 Thread GitBox
shwstppr merged PR #6080: URL: https://github.com/apache/cloudstack/pull/6080 -- 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:

[GitHub] [cloudstack] shwstppr commented on issue #6633: Affinity group creation sometimes fails due to race condition

2022-08-12 Thread GitBox
shwstppr commented on issue #6633: URL: https://github.com/apache/cloudstack/issues/6633#issuecomment-1213380169 @mrog that change is already there in 4.17.0 https://github.com/apache/cloudstack/commit/293517ad728 You will have to upgrade or create your own packages for 4.14 -- This

[GitHub] [cloudstack] mrog commented on issue #6633: Affinity group creation sometimes fails due to race condition

2022-08-12 Thread GitBox
mrog commented on issue #6633: URL: https://github.com/apache/cloudstack/issues/6633#issuecomment-1213368338 If ApiServer.getBaseAsyncCreateResponse() was able to find jobs that have been removed, that would fix this issue. But I don't know if it would cause other problems. Is there a

[GitHub] [cloudstack] mrog commented on issue #6633: Affinity group creation sometimes fails due to race condition

2022-08-12 Thread GitBox
mrog commented on issue #6633: URL: https://github.com/apache/cloudstack/issues/6633#issuecomment-1213332061 @shwstppr It might be hard to reproduce. Creating affinity groups usually works, but one of our developers saw this issue 1 out of 4 tries yesterday. We have the CloudStack

[GitHub] [cloudstack] blueorangutan commented on pull request #6080: Increase size of column 'value' at table 'account_details'

2022-08-12 Thread GitBox
blueorangutan commented on PR #6080: URL: https://github.com/apache/cloudstack/pull/6080#issuecomment-1213315007 Trillian test result (tid-4675) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40261 seconds Marvin logs:

[GitHub] [cloudstack] blueorangutan commented on pull request #6625: vmware,cks: fix attachiso failure with vmware drs

2022-08-12 Thread GitBox
blueorangutan commented on PR #6625: URL: https://github.com/apache/cloudstack/pull/6625#issuecomment-1213300134 Trillian test result (tid-4674) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 43065 seconds Marvin logs:

[GitHub] [cloudstack] cdfgallo commented on issue #6626: Compute offering "Custom constrained" not working

2022-08-12 Thread GitBox
cdfgallo commented on issue #6626: URL: https://github.com/apache/cloudstack/issues/6626#issuecomment-1213267042 here attached you can find the log of the vm deploy. [debug.log](https://github.com/apache/cloudstack/files/9324680/debug.log) I trried as RAM value 1024 or 2048 with same

[GitHub] [cloudstack] blueorangutan commented on pull request #6619: synchronise VM creation to guard resource limit

2022-08-12 Thread GitBox
blueorangutan commented on PR #6619: URL: https://github.com/apache/cloudstack/pull/6619#issuecomment-1213225508 Trillian test result (tid-4677) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 20073 seconds Marvin logs:

[GitHub] [cloudstack] tungntvn23 commented on issue #4314: Failed to attach binaries ISO for VM when create kubernetes cluster

2022-08-12 Thread GitBox
tungntvn23 commented on issue #4314: URL: https://github.com/apache/cloudstack/issues/4314#issuecomment-1213208235 > @tungntvn23 From your original logs what I understand is that ISO is failing to get attached because ACS is not being able to find VM on the given host to execute

[GitHub] [cloudstack] tungntvn23 commented on issue #4314: Failed to attach binaries ISO for VM when create kubernetes cluster

2022-08-12 Thread GitBox
tungntvn23 commented on issue #4314: URL: https://github.com/apache/cloudstack/issues/4314#issuecomment-1213205251 > @tungntvn23 can you share your CloudStack version, hypervisor and storage you're using and CKS/kubernetes iso/version? cc @shwstppr @davidjumani Hi @rohityadavcloud ,

[GitHub] [cloudstack] codecov[bot] commented on pull request #6589: [Veeam] disable jobs but keep backups

2022-08-12 Thread GitBox
codecov[bot] commented on PR #6589: URL: https://github.com/apache/cloudstack/pull/6589#issuecomment-1213133212 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6589?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)

[GitHub] [cloudstack] nvazquez commented on pull request #6078: Fix hv_ss_reserve value for volumes on migration

2022-08-12 Thread GitBox
nvazquez commented on PR #6078: URL: https://github.com/apache/cloudstack/pull/6078#issuecomment-1213101428 @shwstppr it is not ready, still needs some work, will try to find some time for it and will update this PR -- This is an automated message from the Apache Git Service. To respond

[GitHub] [cloudstack] shwstppr commented on issue #6633: Affinity group creation sometimes fails due to race condition

2022-08-12 Thread GitBox
shwstppr commented on issue #6633: URL: https://github.com/apache/cloudstack/issues/6633#issuecomment-1213072722 @mrog how to reproduce this? I tested with 4.17 branch env, trying to create 50 groups simultaneously and it seem to work okay with following script using cmk, ```

svn commit: r56257 - in /release/cloudstack/releases: 4.16.1.1/apache-cloudstack-4.16.1.1-src.tar.bz2.sha512 4.17.0.1/apache-cloudstack-4.17.0.1-src.tar.bz2.sha512

2022-08-12 Thread rohit
Author: rohit Date: Fri Aug 12 11:59:56 2022 New Revision: 56257 Log: Fix SHA512 checksums for Apache CloudStack 4.16.1.1 and 4.17.0.1 src tarball Modified: release/cloudstack/releases/4.16.1.1/apache-cloudstack-4.16.1.1-src.tar.bz2.sha512

[GitHub] [cloudstack] blueorangutan commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
blueorangutan commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1213026678 @shwstppr a 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

[GitHub] [cloudstack] shwstppr commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
shwstppr commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1213026300 @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

[GitHub] [cloudstack] shwstppr closed issue #6621: Host memory stats are not updated

2022-08-12 Thread GitBox
shwstppr closed issue #6621: Host memory stats are not updated URL: https://github.com/apache/cloudstack/issues/6621 -- 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

[GitHub] [cloudstack] shwstppr commented on issue #6621: Host memory stats are not updated

2022-08-12 Thread GitBox
shwstppr commented on issue #6621: URL: https://github.com/apache/cloudstack/issues/6621#issuecomment-1213025716 Fixed with #6622 -- 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

[cloudstack] branch 4.17 updated: kvm: update host memory stats (#6622)

2022-08-12 Thread shwstppr
This is an automated email from the ASF dual-hosted git repository. shwstppr pushed a commit to branch 4.17 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.17 by this push: new 696b93f421 kvm: update host memory stats

[GitHub] [cloudstack] shwstppr merged pull request #6622: Fixes #6621 - Update host memory stats

2022-08-12 Thread GitBox
shwstppr merged PR #6622: URL: https://github.com/apache/cloudstack/pull/6622 -- 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:

[GitHub] [cloudstack] rohityadavcloud opened a new issue, #6637: Use Calico or Cilium in CKS

2022-08-12 Thread GitBox
rohityadavcloud opened a new issue, #6637: URL: https://github.com/apache/cloudstack/issues/6637 Weave project are looking for maintainers, it may be worth exploring what CNI is widely used and standard/stable for CKS use-case. # ISSUE TYPE * Improvement Request #

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944290239 ## agent/src/main/java/com/cloud/agent/properties/AgentProperties.java: ## @@ -24,43 +24,710 @@ * @param type of the default value. */ public class

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944287338 ## agent/src/main/java/com/cloud/agent/AgentShell.java: ## @@ -265,75 +266,80 @@ protected boolean parseCommand(final String[] args) throws

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944289262 ## agent/src/main/java/com/cloud/agent/AgentShell.java: ## @@ -492,7 +488,7 @@ public void start() { String instance = getProperty(null,

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944287338 ## agent/src/main/java/com/cloud/agent/AgentShell.java: ## @@ -265,75 +266,80 @@ protected boolean parseCommand(final String[] args) throws

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944282045 ## agent/conf/agent.properties: ## @@ -17,277 +17,438 @@ # Sample configuration file for CloudStack agent -# The GUID to identify the agent with, this is

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944275016 ## agent/src/test/java/com/cloud/agent/properties/AgentPropertiesTest.java: ## @@ -0,0 +1,41 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [cloudstack] DaanHoogland commented on a diff in pull request #6348: Improving code related to the Agent properties

2022-08-12 Thread GitBox
DaanHoogland commented on code in PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#discussion_r944270970 ## agent/src/test/java/com/cloud/agent/AgentShellTest.java: ## @@ -59,4 +84,290 @@ public void testGetHost() { }

[GitHub] [cloudstack] sonarcloud[bot] commented on pull request #6619: synchronise VM creation to guard resource limit

2022-08-12 Thread GitBox
sonarcloud[bot] commented on PR #6619: URL: https://github.com/apache/cloudstack/pull/6619#issuecomment-1212894656 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [cloudstack] blueorangutan commented on pull request #6630: Fix delete parent snapshot

2022-08-12 Thread GitBox
blueorangutan commented on PR #6630: URL: https://github.com/apache/cloudstack/pull/6630#issuecomment-1212889848 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3970 -- This is an automated message from the

[GitHub] [cloudstack] rohityadavcloud closed issue #6635: Can not deploy VM using libvirt-8.0.0 ?

2022-08-12 Thread GitBox
rohityadavcloud closed issue #6635: Can not deploy VM using libvirt-8.0.0 ? URL: https://github.com/apache/cloudstack/issues/6635 -- 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

[GitHub] [cloudstack] rohityadavcloud commented on issue #6635: Can not deploy VM using libvirt-8.0.0 ?

2022-08-12 Thread GitBox
rohityadavcloud commented on issue #6635: URL: https://github.com/apache/cloudstack/issues/6635#issuecomment-1212886290 @xuanyuanaosheng you can either upgrade to latest 4.17 release (4.17.0.1, 4.17.0.0) or use SB customer patches which fixes this libvirt-8 issue, read more -

[GitHub] [cloudstack] rohityadavcloud commented on issue #6628: Harvester HCI Compatibility

2022-08-12 Thread GitBox
rohityadavcloud commented on issue #6628: URL: https://github.com/apache/cloudstack/issues/6628#issuecomment-1212884338 @RagingAdmin it depends, if they're using libvirtd etc then it should be compatible. CloudStack already supports openSUSE Leap 15 as KVM host. Why don't you give it a

[GitHub] [cloudstack] codecov[bot] commented on pull request #6636: ui: reset disksize param on offering change in scale vm

2022-08-12 Thread GitBox
codecov[bot] commented on PR #6636: URL: https://github.com/apache/cloudstack/pull/6636#issuecomment-1212869965 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6636?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)

[GitHub] [cloudstack] blueorangutan commented on pull request #6619: synchronise VM creation to guard resource limit

2022-08-12 Thread GitBox
blueorangutan commented on PR #6619: URL: https://github.com/apache/cloudstack/pull/6619#issuecomment-1212866196 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3969 -- This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on pull request #6630: Fix delete parent snapshot

2022-08-12 Thread GitBox
blueorangutan commented on PR #6630: URL: https://github.com/apache/cloudstack/pull/6630#issuecomment-1212850790 @DaanHoogland a 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.

[GitHub] [cloudstack] DaanHoogland commented on pull request #6630: Fix delete parent snapshot

2022-08-12 Thread GitBox
DaanHoogland commented on PR #6630: URL: https://github.com/apache/cloudstack/pull/6630#issuecomment-1212850294 @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

[GitHub] [cloudstack] github-actions[bot] commented on pull request #6522: Volume encryption feature

2022-08-12 Thread GitBox
github-actions[bot] commented on PR #6522: URL: https://github.com/apache/cloudstack/pull/6522#issuecomment-1212832899 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

[cloudstack] branch main updated: Improve log messages on VolumeOrchestrator class (#6408)

2022-08-12 Thread 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 9717ed9af2 Improve log messages on

[GitHub] [cloudstack] DaanHoogland merged pull request #6408: Improve log messages on VolumeOrchestrator class

2022-08-12 Thread GitBox
DaanHoogland merged PR #6408: URL: https://github.com/apache/cloudstack/pull/6408 -- 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:

[GitHub] [cloudstack] DaanHoogland commented on issue #6628: Harvester HCI Compatibility

2022-08-12 Thread GitBox
DaanHoogland commented on issue #6628: URL: https://github.com/apache/cloudstack/issues/6628#issuecomment-1212801816 It can be tried to be added, there will have to be some scripting done for it to work, probably, @RagingAdmin . -- This is an automated message from the Apache Git

[GitHub] [cloudstack] DaanHoogland commented on issue #6601: apache apidocs FAILURE in Build From Source

2022-08-12 Thread GitBox
DaanHoogland commented on issue #6601: URL: https://github.com/apache/cloudstack/issues/6601#issuecomment-1212800475 > > Hi @ahmadamirahmadi007 can you create a python alias for python3? > after create alias for both python3 and python2 and run again i encountered with this error: > >

[GitHub] [cloudstack] DaanHoogland commented on issue #3693: cleanup test resources

2022-08-12 Thread GitBox
DaanHoogland commented on issue #3693: URL: https://github.com/apache/cloudstack/issues/3693#issuecomment-1212796563 moving to the next milestone @shwstppr even if I may do some files. I will attach the PR to a feasible milestone. -- This is an automated message from the Apache Git

[GitHub] [cloudstack] codecov[bot] commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
codecov[bot] commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1212796409 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6634?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)

[GitHub] [cloudstack] blueorangutan commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
blueorangutan commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1212794090 Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3968 -- This is an automated message from the

[GitHub] [cloudstack] vdombrovski commented on issue #6620: Created tags are not deleted whenever a public IP is disassociated

2022-08-12 Thread GitBox
vdombrovski commented on issue #6620: URL: https://github.com/apache/cloudstack/issues/6620#issuecomment-1212780974 To add to the discussion, public IPs are currently the only shared resource that can be tagged. The issue does not arise for other resource types because those will never

[GitHub] [cloudstack] blueorangutan commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
blueorangutan commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1212770645 @shwstppr a 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. --

[GitHub] [cloudstack] shwstppr commented on pull request #6634: server: fix delete resource tag permission

2022-08-12 Thread GitBox
shwstppr commented on PR #6634: URL: https://github.com/apache/cloudstack/pull/6634#issuecomment-1212770321 @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

[GitHub] [cloudstack] shwstppr commented on issue #6626: Compute offering "Custom constrained" not working

2022-08-12 Thread GitBox
shwstppr commented on issue #6626: URL: https://github.com/apache/cloudstack/issues/6626#issuecomment-1212759674 @cdfgallo I'm not able to reproduce in my lab. Can you please share logs of the complete deploy VM action? What value of RAM was VM provided? -- This is an automated

[GitHub] [cloudstack] blueorangutan commented on pull request #6636: ui: reset disksize param on offering change in scale vm

2022-08-12 Thread GitBox
blueorangutan commented on PR #6636: URL: https://github.com/apache/cloudstack/pull/6636#issuecomment-1212758189 UI build: :heavy_check_mark: Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6636 (SL-JID-2134) -- This is an automated message from the Apache Git Service. To