Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2570487755 Sorry @nvazquez , I was a bit eager on this one, let's revert. cc @vishesh92 @jbampton @nvazquez , do you see a possibility to make this check work (by options on the check or

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2025-01-03 Thread via GitHub
nvazquez commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2570042812 Hi guys @jbampton @DaanHoogland @vishesh92 I'm not able to get CKS clusters into Running state after these changes (at least after merging main branch on my PR #9102): ![image](ht

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-31 Thread via GitHub
DaanHoogland merged PR #9133: URL: https://github.com/apache/cloudstack/pull/9133 -- 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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-30 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2566114545 [SF] Trillian test result (tid-12003) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 34 seconds Marvin logs: https://github.com

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-29 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564832513 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-29 Thread via GitHub
DaanHoogland commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564832235 @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 co

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-29 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564806158 [SF] Trillian test result (tid-12001) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 109701 seconds Marvin logs: https://github.com

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-28 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564299151 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-28 Thread via GitHub
DaanHoogland commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564298849 @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 co

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-28 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564290466 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11928 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-28 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564277783 @jbampton 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.

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-28 Thread via GitHub
jbampton commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2564277500 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-20 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2557887929 Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 11870 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-20 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2557774390 @jbampton 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.

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-12-20 Thread via GitHub
jbampton commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2557772381 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-29 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2443757893 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11466 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-28 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2442072027 Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 11460 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-28 Thread via GitHub
DaanHoogland commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2441992814 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-28 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2441998983 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-25 Thread via GitHub
vishesh92 commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2436828301 @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 comme

[PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-23 Thread via GitHub
jbampton opened a new pull request, #9133: URL: https://github.com/apache/cloudstack/pull/9133 https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#check-yaml ### Description This PR adds another check or test to our pre-commit framework. From the official pr

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-23 Thread via GitHub
DaanHoogland closed pull request #9133: pre-commit: add hook `check-yaml` URL: https://github.com/apache/cloudstack/pull/9133 -- 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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-14 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2411362408 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11334 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-14 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2411127295 @jbampton 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.

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-10-14 Thread via GitHub
jbampton commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2411125852 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-07-07 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2212925448 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10279 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-07-07 Thread via GitHub
blueorangutan commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2212858455 @jbampton 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.

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-07-07 Thread via GitHub
jbampton commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2212857128 @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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-06-03 Thread via GitHub
DaanHoogland closed pull request #9133: pre-commit: add hook `check-yaml` URL: https://github.com/apache/cloudstack/pull/9133 -- 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

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-05-29 Thread via GitHub
jbampton commented on code in PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#discussion_r1618968734 ## .pre-commit-config.yaml: ## @@ -34,7 +34,8 @@ repos: - id: check-merge-conflict - id: check-symlinks - id: check-vcs-permalinks - #- id

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-05-29 Thread via GitHub
jbampton commented on code in PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#discussion_r1618959074 ## plugins/integrations/kubernetes-service/src/main/resources/conf/k8s-node.yml: ## @@ -21,8 +21,8 @@ users: - name: cloud sudo: ALL=(ALL) NOPASSWD:ALL

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-05-29 Thread via GitHub
DaanHoogland commented on code in PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#discussion_r1618449412 ## .pre-commit-config.yaml: ## @@ -34,7 +34,8 @@ repos: - id: check-merge-conflict - id: check-symlinks - id: check-vcs-permalinks - #

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-05-28 Thread via GitHub
codecov[bot] commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2135408922 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9133?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

[PR] pre-commit: add hook `check-yaml` [cloudstack]

2024-05-28 Thread via GitHub
jbampton opened a new pull request, #9133: URL: https://github.com/apache/cloudstack/pull/9133 https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#check-yaml ### Description This PR adds another check or test to our pre-commit framework. From the official pr