BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2167950272
> @BryanMLima shouldn't we need to run smoketests on the PR, or do we think
they were covered by GHA/simulator smoketests?
@rohityadavcloud, the smoke tests were already run i
rohityadavcloud commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2167357934
@BryanMLima shouldn't we need to run smoketests on the PR, or do we think
they were covered by GHA/simulator smoketests?
--
This is an automated message from the Apache Git S
BryanMLima merged PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372
--
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...@clou
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2166648210
Merging this one based on reviews and manual tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2163750397
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9906
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2163630337
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2163628305
@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
BryanMLima closed pull request #8372: [Quota] Add API to list preset variables
URL: https://github.com/apache/cloudstack/pull/8372
--
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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2138172862
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9733
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2138053546
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2138051406
@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
BryanMLima opened a new pull request, #8372:
URL: https://github.com/apache/cloudstack/pull/8372
### Description
This PR extends the functionality introduced in PR #5909, allowing users to
list the preset variables via API, facilitating the creation of the activation
rules while crea
DaanHoogland closed pull request #8372: [Quota] Add API to list preset variables
URL: https://github.com/apache/cloudstack/pull/8372
--
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
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2135199283
@bernardodemarco, thanks for testing it.
> @BryanMLima will you apply @bernardodemarco 's sugestion [#8372
(comment)](https://github.com/apache/cloudstack/pull/8372#discussion
DaanHoogland commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2134508800
@BryanMLima will you apply @bernardodemarco 's sugestion
https://github.com/apache/cloudstack/pull/8372#discussion_r1616409198 , or can
we merge as is?
Also, @BryanMLima th
bernardodemarco commented on code in PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#discussion_r1616409198
##
framework/quota/src/main/java/org/apache/cloudstack/quota/activationrule/presetvariables/Value.java:
##
@@ -23,25 +23,75 @@
import com.cloud.storage.Snap
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2067146592
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9338
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2067028881
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2067027893
@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
github-actions[bot] commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2003895988
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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1971709919
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8821
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1971612590
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1971610519
@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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1936849088
[SF] Trillian test result (tid-9148)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41759 seconds
Marvin logs:
https://github.com
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1936189551
@DaanHoogland 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 r
DaanHoogland commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1936187112
@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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1934291387
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8590
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1934160148
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1934159524
@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
github-actions[bot] commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1934110503
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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1906612149
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8423
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1906487901
@BryanMLima 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.
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1906485099
@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
BryanMLima commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1906484675
> clgtm, and lots of test. Just a thought; an integration test seems
appropriate for such a profoundly user facing feature, would you say
@BryanMLima ?
@DaanHoogland, got bac
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1869170367
[SF] Trillian test result (tid-8680)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42187 seconds
Marvin logs:
https://github.com
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1868966158
@sureshanaparti 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
sureshanaparti commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1868965628
@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
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1868944790
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8141
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
blueorangutan commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1868920251
@sureshanaparti 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
sureshanaparti commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1868919826
@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
sureshanaparti commented on code in PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#discussion_r1431434349
##
plugins/database/quota/src/main/java/org/apache/cloudstack/api/response/QuotaPresetVariablesItemResponse.java:
##
@@ -0,0 +1,47 @@
+//Licensed to the Apach
codecov[bot] commented on PR #8372:
URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-1860832271
##
[Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8372?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
BryanMLima opened a new pull request, #8372:
URL: https://github.com/apache/cloudstack/pull/8372
### Description
This PR extends the functionality introduced in PR #5909, allowing users to
list the preset variables via API, facilitating the creation of the activation
rules while crea
43 matches
Mail list logo
