Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

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

Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #9230: URL: https://github.com/apache/cloudstack/pull/9230#issuecomment-2164023360 [SF] Trillian test result (tid-10418) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42973 seconds Marvin logs:

Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

2024-06-12 Thread via GitHub
vishesh92 commented on PR #9230: URL: https://github.com/apache/cloudstack/pull/9230#issuecomment-2162660050 @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] Ensure marvin package version is PEP440 compliant [cloudstack]

2024-06-12 Thread via GitHub
vishesh92 commented on PR #9230: URL: https://github.com/apache/cloudstack/pull/9230#issuecomment-2162661364 > clgtm, not sure though if it makes sense to change the version globally this way instead of in the marvin setup. We are changing only for marvin here. I am not sure if

Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

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

Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

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

Re: [PR] Ensure marvin package version is PEP440 compliant [cloudstack]

2024-06-12 Thread via GitHub
codecov[bot] commented on PR #9230: URL: https://github.com/apache/cloudstack/pull/9230#issuecomment-2162354828 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9230?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report