[cloudstack] branch main updated: make api rate limit test more robust (#6984)

2022-12-15 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 162af93e114 make api rate limit test more robust (

[GitHub] [cloudstack] DaanHoogland closed issue #6975: ApiRateLimitTest.canDoReasonableNumberOfApiAccessPerSecond fails randomly in nested envs

2022-12-15 Thread GitBox
DaanHoogland closed issue #6975: ApiRateLimitTest.canDoReasonableNumberOfApiAccessPerSecond fails randomly in nested envs URL: https://github.com/apache/cloudstack/issues/6975 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack] DaanHoogland merged pull request #6984: make api rate limit test a little more robust

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354322875 @rohityadavcloud 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] rohityadavcloud commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354322275 @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 speci

[GitHub] [cloudstack] blueorangutan commented on pull request #6803: ui,server,api: resource metrics improvements

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6426: Configurable MTU for VR

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6803: ui,server,api: resource metrics improvements

2022-12-15 Thread GitBox
blueorangutan commented on PR #6803: URL: https://github.com/apache/cloudstack/pull/6803#issuecomment-1354279263 @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. -- T

[GitHub] [cloudstack] shwstppr commented on pull request #6803: ui,server,api: resource metrics improvements

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354265501 Trillian Build Failed (tid-5509) -- 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 t

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354262509 Trillian Build Failed (tid-5508) -- 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 t

[GitHub] [cloudstack] blueorangutan commented on pull request #6426: Configurable MTU for VR

2022-12-15 Thread GitBox
blueorangutan commented on PR #6426: URL: https://github.com/apache/cloudstack/pull/6426#issuecomment-1354259569 @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. -- T

[GitHub] [cloudstack] shwstppr commented on pull request #6426: Configurable MTU for VR

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354259049 @rohityadavcloud a Trillian-Jenkins test job (rocky8 mgmt + kvm-rocky8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respo

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354258634 @blueorangutan test rocky8 kvm-rocky8 -- 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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354258552 @rohityadavcloud a Trillian-Jenkins test job (suse15 mgmt + kvm-suse15) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respo

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354258354 @blueorangutan test suse15 kvm-suse15 -- 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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354258036 @rohityadavcloud I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv] Mgmt OS options: [

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1354257439 Debian failure caused by backend Jenkins job issue, not related to this PR. EL7 can be merged with centos8 pkging but on EL7 the rpm/rpmbuild versions are less than 4.13 whic

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353740470 Packaging result: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 4970 -- This is an automated message fro

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353736093 @rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is an a

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge suse15 to el8

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353734862 @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

[GitHub] [cloudstack] github-actions[bot] commented on pull request #6803: ui,server,api: resource metrics improvements

2022-12-15 Thread GitBox
github-actions[bot] commented on PR #6803: URL: https://github.com/apache/cloudstack/pull/6803#issuecomment-1353733853 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 Gi

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353719674 @weizhouapache 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] weizhouapache commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
weizhouapache commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353718826 @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 specifi

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353714152 @rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is an a

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353713187 @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

[GitHub] [cloudstack] blueorangutan commented on pull request #6803: ui,server,api: resource metrics improvements

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6803: ui,server,api: resource metrics improvements

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

[GitHub] [cloudstack] stephankruggg commented on pull request #6925: Add and improve logs in snapshot scheduling

2022-12-15 Thread GitBox
stephankruggg commented on PR #6925: URL: https://github.com/apache/cloudstack/pull/6925#issuecomment-1353577257 > @stephankruggg , i still oppose to unguarded logging with string evaluation in parameters. i have not seen arguments to counter the downside of it yet. So please look at #6973

[cloudstack] branch main updated: Improves CPU usage info (#6784)

2022-12-15 Thread gutoveronezi
This is an automated email from the ASF dual-hosted git repository. gutoveronezi 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 fc5bd85ecbf Improves CPU usage info (#6784

[GitHub] [cloudstack] GutoVeronezi merged pull request #6784: Improves CPU usage info

2022-12-15 Thread GitBox
GutoVeronezi merged PR #6784: URL: https://github.com/apache/cloudstack/pull/6784 -- 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...@cl

[GitHub] [cloudstack] GutoVeronezi commented on pull request #6784: Improves CPU usage info

2022-12-15 Thread GitBox
GutoVeronezi commented on PR #6784: URL: https://github.com/apache/cloudstack/pull/6784#issuecomment-1353536785 Merging this one based on the approvals and test results. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [cloudstack] RodrigoDLopez commented on pull request #6505: Gives the possibility to redirect to external links when the property is defined

2022-12-15 Thread GitBox
RodrigoDLopez commented on PR #6505: URL: https://github.com/apache/cloudstack/pull/6505#issuecomment-1353496992 @DaanHoogland, @rohityadavcloud, @utchoang and @shwstppr is there any suggestion or request on this or are we ready to go? -- This is an automated message from the Ap

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

2022-12-15 Thread GitBox
GutoVeronezi commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1353487861 @DaanHoogland I think I found out what was happening. When patching the system VMs, the file `systemvm/agent/conf/agent.properties` is copied to them. This file contains two hardc

[GitHub] [cloudstack] blueorangutan commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
blueorangutan commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1353423018 Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4969 -- This is an automated

[GitHub] [cloudstack] blueorangutan commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
blueorangutan commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1353380410 @NuxRo 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. -- This

[GitHub] [cloudstack] NuxRo commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
NuxRo commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1353379951 @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 commen

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353324009 @rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is an a

[GitHub] [cloudstack] rohityadavcloud commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353323536 @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

[GitHub] [cloudstack] blueorangutan commented on pull request #6870: kvm: correctly set vm cpu topology

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

[GitHub] [cloudstack] codecov[bot] commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
codecov[bot] commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353322435 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6996?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+So

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

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

[GitHub] [cloudstack] codecov[bot] commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
codecov[bot] commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353275814 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6995?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+So

[GitHub] [cloudstack] blueorangutan commented on pull request #6870: kvm: correctly set vm cpu topology

2022-12-15 Thread GitBox
blueorangutan commented on PR #6870: URL: https://github.com/apache/cloudstack/pull/6870#issuecomment-1353256398 @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. -- T

[GitHub] [cloudstack] shwstppr commented on pull request #6870: kvm: correctly set vm cpu topology

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
blueorangutan commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353249253 @nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack] nvazquez commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
nvazquez commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353248940 @blueorangutan test centos7 vmware-67u3 -- 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

[GitHub] [cloudstack] sonarcloud[bot] commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
sonarcloud[bot] commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353227512 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Q

[GitHub] [cloudstack] blueorangutan commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

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

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

2022-12-15 Thread GitBox
blueorangutan commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1353204790 @GutoVeronezi 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] GutoVeronezi commented on pull request #6348: Improving code related to the Agent properties

2022-12-15 Thread GitBox
GutoVeronezi commented on PR #6348: URL: https://github.com/apache/cloudstack/pull/6348#issuecomment-1353204381 @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] codecov[bot] commented on pull request #6994: ui: fix persist networks config in deploy vm form

2022-12-15 Thread GitBox
codecov[bot] commented on PR #6994: URL: https://github.com/apache/cloudstack/pull/6994#issuecomment-1353159282 # [Codecov](https://codecov.io/gh/apache/cloudstack/pull/6994?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+So

[GitHub] [cloudstack] blueorangutan commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
blueorangutan commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353138838 @nvazquez 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. -- T

[GitHub] [cloudstack] nvazquez commented on a diff in pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
nvazquez commented on code in PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#discussion_r1049673476 ## utils/src/main/java/com/cloud/utils/nicira/nvp/plugin/NiciraNvpApiVersion.java: ## @@ -40,8 +40,10 @@ public static synchronized boolean isApiVersionLowerThan(S

[GitHub] [cloudstack] nvazquez commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
nvazquez commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353137384 @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 com

[GitHub] [cloudstack] sonarcloud[bot] commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
sonarcloud[bot] commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1353124618 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Q

[GitHub] [cloudstack] blueorangutan commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1353033164 Packaging result: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_multiplication_x: debian :heavy_check_mark: suse15. SL-JID 4963 -- This is an automated message fro

[GitHub] [cloudstack] stephankruggg commented on a diff in pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
stephankruggg commented on code in PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#discussion_r1049614963 ## utils/src/main/java/com/cloud/utils/nicira/nvp/plugin/NiciraNvpApiVersion.java: ## @@ -40,8 +40,10 @@ public static synchronized boolean isApiVersionLowerT

[GitHub] [cloudstack] blueorangutan commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
blueorangutan commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-135311 @nvazquez 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. -- T

[GitHub] [cloudstack] nvazquez commented on pull request #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
nvazquez commented on PR #6996: URL: https://github.com/apache/cloudstack/pull/6996#issuecomment-1352999110 @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 com

[GitHub] [cloudstack] shwstppr commented on pull request #6426: Configurable MTU for VR

2022-12-15 Thread GitBox
shwstppr commented on PR #6426: URL: https://github.com/apache/cloudstack/pull/6426#issuecomment-1352992816 This would need a new systemvm template cc @rohityadavcloud @weizhouapache -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [cloudstack] blueorangutan commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
blueorangutan commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1352992703 @rohityadavcloud 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] rohityadavcloud commented on pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
rohityadavcloud commented on PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#issuecomment-1352991881 @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 speci

[GitHub] [cloudstack] blueorangutan commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
blueorangutan commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1352992281 Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4962 -- This is an automated

[GitHub] [cloudstack] nvazquez opened a new pull request, #6996: [VMware] Remove unnecessary logs on VM deployments

2022-12-15 Thread GitBox
nvazquez opened a new pull request, #6996: URL: https://github.com/apache/cloudstack/pull/6996 ### Description This PR removes unnecessary logging on VMware deployments in which the NVP plugin is not enabled. On each StartCommand a similar line is logged even if the NVP plugin is not

[GitHub] [cloudstack] rohityadavcloud commented on a diff in pull request #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
rohityadavcloud commented on code in PR #6995: URL: https://github.com/apache/cloudstack/pull/6995#discussion_r1049579891 ## packaging/centos7: ## @@ -0,0 +1 @@ +rpm Review Comment: this is symlink btw -- This is an automated message from the Apache Git Service. To respo

[GitHub] [cloudstack] rohityadavcloud opened a new pull request, #6995: Packages consolidation: merge el7, el8, suse15

2022-12-15 Thread GitBox
rohityadavcloud opened a new pull request, #6995: URL: https://github.com/apache/cloudstack/pull/6995 Using the boolean operator we can create rpm pkgs that work on all support distros - el7, el8 and suse15. -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack] nvazquez commented on pull request #6919: Minor console access refactor and improvements

2022-12-15 Thread GitBox
nvazquez commented on PR #6919: URL: https://github.com/apache/cloudstack/pull/6919#issuecomment-1352973762 @vladimirpetrov I have addressed @weizhouapache's comments, could you review again please? -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [cloudstack] StepBee commented on issue #6807: Custom Constrained Service Offering vs Qemu - CPU topology doesn't match maximum vcpu count

2022-12-15 Thread GitBox
StepBee commented on issue #6807: URL: https://github.com/apache/cloudstack/issues/6807#issuecomment-1352966236 I can confirm the patch is working. Many thanks for your efforts on this one! Finally i had to dig into building cloudstack from source - a valuable learning ;-) --

[GitHub] [cloudstack] StepBee commented on pull request #6870: kvm: correctly set vm cpu topology

2022-12-15 Thread GitBox
StepBee commented on PR #6870: URL: https://github.com/apache/cloudstack/pull/6870#issuecomment-1352958117 @shwstppr @DaanHoogland Finally was able to build from source, install and test in our testing environment. I can confirm it's working. Many thanks for your efforts on thi

[GitHub] [cloudstack] blueorangutan commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
blueorangutan commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1352956391 @harikrishna-patnala a Jenkins job has been kicked to build packages. It will be bundled with SystemVM template(s). I'll keep you posted as I make progress. -- This is

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
harikrishna-patnala commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1352955546 @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

[GitHub] [cloudstack] shwstppr opened a new pull request, #6994: ui: fix persist networks config in deploy vm form

2022-12-15 Thread GitBox
shwstppr opened a new pull request, #6994: URL: https://github.com/apache/cloudstack/pull/6994 ### Description Fixes #6974 ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking cha

[GitHub] [cloudstack] blueorangutan commented on pull request #6840: infra: edge zones

2022-12-15 Thread GitBox
blueorangutan commented on PR #6840: URL: https://github.com/apache/cloudstack/pull/6840#issuecomment-1352824259 Trillian Build Failed (tid-5505) -- 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 t

[GitHub] [cloudstack] blueorangutan commented on pull request #6840: infra: edge zones

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6426: Configurable MTU for VR

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6840: infra: edge zones

2022-12-15 Thread GitBox
blueorangutan commented on PR #6840: URL: https://github.com/apache/cloudstack/pull/6840#issuecomment-1352743712 @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. -- T

[GitHub] [cloudstack] shwstppr commented on pull request #6840: infra: edge zones

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6840: infra: edge zones

2022-12-15 Thread GitBox
blueorangutan commented on PR #6840: URL: https://github.com/apache/cloudstack/pull/6840#issuecomment-1352726390 Trillian Build Failed (tid-5504) -- 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 t

[GitHub] [cloudstack] borisstoyanov opened a new issue, #6993: Root volume of system vms is being listed

2022-12-15 Thread GitBox
borisstoyanov opened a new issue, #6993: URL: https://github.com/apache/cloudstack/issues/6993 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ API/UI ~~~ # CLOUDSTACK VERSION ~~~ 4.18 [main] ~~~ # CONFIGURAT

[GitHub] [cloudstack] levindecaro commented on issue #6794: Cannot start cloudstack-agent after upgrade from 4.16.1 to 4.17.1

2022-12-15 Thread GitBox
levindecaro commented on issue #6794: URL: https://github.com/apache/cloudstack/issues/6794#issuecomment-1352721193 @DaanHoogland I'll try to validate it soon by reinstalling the agent with this revised version, I'm not sure it is 100% reproducible, but i'll report the result if got anythin

[GitHub] [cloudstack] levindecaro closed issue #6796: config.json wrongly set to 640 after upgrade to 4.17.1

2022-12-15 Thread GitBox
levindecaro closed issue #6796: config.json wrongly set to 640 after upgrade to 4.17.1 URL: https://github.com/apache/cloudstack/issues/6796 -- 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 speci

[GitHub] [cloudstack] levindecaro commented on issue #6796: config.json wrongly set to 640 after upgrade to 4.17.1

2022-12-15 Thread GitBox
levindecaro commented on issue #6796: URL: https://github.com/apache/cloudstack/issues/6796#issuecomment-1352718006 @DaanHoogland, i'm pretty sure i have to change config.json to 644 to make ACS accessible in the upgrade exercise, i have screen capture with forbidden error. Since i fixed th

[GitHub] [cloudstack] blueorangutan commented on pull request #6426: Configurable MTU for VR

2022-12-15 Thread GitBox
blueorangutan commented on PR #6426: URL: https://github.com/apache/cloudstack/pull/6426#issuecomment-1352709330 @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. -- T

[GitHub] [cloudstack] shwstppr commented on pull request #6426: Configurable MTU for VR

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

[GitHub] [cloudstack] blueorangutan commented on pull request #6924: [Draft] User two factor authentication

2022-12-15 Thread GitBox
blueorangutan commented on PR #6924: URL: https://github.com/apache/cloudstack/pull/6924#issuecomment-1352699160 Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 4958 -- This is an automated