Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-07-29 Thread via GitHub
github-actions[bot] commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-3132581400 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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-07-28 Thread via GitHub
DaanHoogland commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r2235600791 ## .github/linters/codespell.txt: ## @@ -12,10 +12,8 @@ afrer afterall againt ags -aktive algoritm allo -alloacate allocted Review Comment: it has o

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-07-28 Thread via GitHub
sureshanaparti commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r2235468375 ## .github/linters/codespell.txt: ## @@ -12,10 +12,8 @@ afrer afterall againt ags -aktive algoritm allo -alloacate allocted Review Comment: ```sug

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-16 Thread via GitHub
github-actions[bot] commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2975364083 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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-12 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2966196495 [SF] Trillian test result (tid-13501) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 60771 seconds Marvin logs: https://github.co

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-12 Thread via GitHub
vishesh92 commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2965359782 @DaanHoogland @jbampton Since we are changing names of the Command classes, the requests might fail on agents until they are upgraded. IMO this requires some upgrade testing to ens

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-11 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2963610157 @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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-11 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2963613803 @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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-09 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2955889823 @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 progre

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-09 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2955886389 @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 specif

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-06-09 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2955886045 @vishesh92 are all your concerns met? -- 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 g

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-28 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2836965692 [SF] Trillian test result (tid-13172) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 54043 seconds Marvin logs: https://github.co

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-28 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2834191359 @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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-28 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2834190372 @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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-26 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2832320330 @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 progre

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-26 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2832319770 @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 specif

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-04-26 Thread via GitHub
DaanHoogland commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2832319701 @vishesh92 , lgty? -- 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 c

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-01-08 Thread via GitHub
github-actions[bot] commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2577831815 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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-01-03 Thread via GitHub
vishesh92 commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r1901553450 ## .github/linters/codespell.txt: ## @@ -35,6 +35,7 @@ aqcuire aqcuired aquire aquiring +assertin Review Comment: @jbampton got it. But can you remoe `fal

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2024-12-31 Thread via GitHub
jbampton commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r1900086633 ## .github/linters/codespell.txt: ## @@ -35,6 +35,7 @@ aqcuire aqcuired aquire aquiring +assertin Review Comment: > I don't see any usages of `assertin`, `

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2024-12-31 Thread via GitHub
vishesh92 commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r196886 ## .github/linters/codespell.txt: ## @@ -35,6 +35,7 @@ aqcuire aqcuired aquire aquiring +assertin Review Comment: I don't see any usages of `assertin`, `c

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2024-12-29 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2564832505 @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 progre

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2024-12-28 Thread via GitHub
blueorangutan commented on PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#issuecomment-2564303629 @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 upgrade codespell and fix spelling [cloudstack]

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

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2024-12-28 Thread via GitHub
jbampton closed pull request #10144: pre-commit upgrade codespell and fix spelling URL: https://github.com/apache/cloudstack/pull/10144 -- 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 c

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

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