abh1sar commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2667927024
> Hi @abh1sar, ok. I will allocate some time to look at this in the coming
days. As I mentioned before, we don't have a bucket level quota setting. We do
have a User level QoS setting.
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2667602018
I got my local build working again and confirmed that indeed we cannot add
buckets currently as quota is now a requirement and create bucket fails with:
> Failed to create bucket wit
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2667339592
Hi @DaanHoogland - please ignore this PR for the moment. I'm going to look
at the plugin again with the new information that @abh1sar has provided
regarding the quota being required.
shwstppr commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1959450513
##
plugins/storage/object/cloudian/pom.xml:
##
@@ -0,0 +1,70 @@
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+
DaanHoogland commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2664940723
@tpodowd , it seems there is some dependency error on debian:
```
09:14:14 [ERROR] [ERROR] Some problems were encountered while processing the
POMs:
09:14:14 [FATAL] Non-
blueorangutan commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2664895967
Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 12496
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
blueorangutan commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2664891493
@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
DaanHoogland commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2664887768
@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
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2664458269
Hi @abh1sar, ok. I will allocate some time to look at this in the coming
days. As I mentioned before, we don't have a bucket level quota setting. We do
have a User level QoS setting. I
abh1sar commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2662507754
Hi @tpodowd ,
Please sync with the latest code and test bucket creation. The quota field
in create bucket api has been made mandatory since
https://github.com/apache/cloudstack/pull
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2661797494
Hi @DaanHoogland @rohityadavcloud - Any update on when this PR is likely to
be merged?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
abh1sar commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2456500473
Thanks @tpodowd for your response.
If we had to implement the functionality to set Account level Quota in
cloudstack, how do you think that could be done?
Does the HyperStore plug
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2456463569
Hi @abh1sar - Thanks for your comment/question.
> I understand Cloudian doesn't support per bucket Quota.
Yes, Cloudian HyperStore does not currently support a bucket stora
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2456596898
Hi @abh1sar. The current plugin does not have any API support itself for
setting Account level QoS (as it only implements the provided plugin APIs). The
HyperStore Admin API itself doe
abh1sar commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2456269992
Hi @tpodowd,
I understand Cloudian doesn't support per bucket Quota.
Does it support per user quota? Is there a way to configure it from within
CloudStack?
cc @rohityadavcl
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1821986268
##
plugins/storage/object/cloudian/README.md:
##
Review Comment:
Ok. Thanks. I updated the README and deleted the `png` file.
--
This is an automated messag
DaanHoogland commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1820454542
##
plugins/storage/object/cloudian/README.md:
##
Review Comment:
fine, but keep in mind the docs are better viewed than any readme in the
code (unless it
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1820231880
##
plugins/storage/object/cloudian/README.md:
##
Review Comment:
Hi @DaanHoogland. I can certainly move the the `png` out. Is the README file
ok (once I remove
DaanHoogland commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1819234469
##
plugins/storage/object/cloudian/README.md:
##
Review Comment:
should this be added to a PR for
https://github.com/apache/cloudstack-documentation?
#
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2425459613
> @tpodowd I did my best with no knowledge of Cloudian 😄 . In any case,
overall it looks good to me. There is only one thing ([#9748
(comment)](https://github.com/apache/cloudstack/pul
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1808040872
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/driver/CloudianHyperStoreObjectStoreDriverImpl.java:
##
@@ -0,0 +1,877 @@
+/
JoaoJandre commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2422278589
> Hi @JoaoJandre - I have pushed another commit to address your review
comments. Thanks for your time and let me know if you have any other concerns
or questions.
@tpodowd I
JoaoJandre commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1806353907
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/driver/CloudianHyperStoreObjectStoreDriverImpl.java:
##
@@ -0,0 +1,877 @@
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2421600614
Hi @JoaoJandre - I have pushed another commit to address your review
comments. Thanks for your time and let me know if you have any other concerns
or questions.
--
This is an automa
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805934773
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/util/CloudianHyperStoreUtil.java:
##
@@ -0,0 +1,218 @@
+// Licensed to the A
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805934147
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/util/CloudianHyperStoreUtil.java:
##
@@ -0,0 +1,218 @@
+// Licensed to the A
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805930584
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/driver/CloudianHyperStoreObjectStoreDriverImpl.java:
##
@@ -0,0 +1,877 @@
+/
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805928197
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/driver/CloudianHyperStoreObjectStoreDriverImpl.java:
##
@@ -0,0 +1,877 @@
+/
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805919446
##
plugins/storage/object/cloudian/src/main/java/org/apache/cloudstack/storage/datastore/driver/CloudianHyperStoreObjectStoreDriverImpl.java:
##
@@ -0,0 +1,877 @@
+/
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805911585
##
plugins/integrations/cloudian/src/main/java/org/apache/cloudstack/cloudian/client/CloudianClient.java:
##
@@ -287,33 +492,42 @@ public CloudianGroup listGroup(fin
tpodowd commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805768948
##
plugins/integrations/cloudian/src/main/java/org/apache/cloudstack/cloudian/client/CloudianClient.java:
##
@@ -159,22 +180,108 @@ private HttpResponse post(final S
JoaoJandre commented on code in PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#discussion_r1805263925
##
plugins/integrations/cloudian/src/main/java/org/apache/cloudstack/cloudian/client/CloudianClient.java:
##
@@ -287,33 +492,42 @@ public CloudianGroup listGroup(
blueorangutan commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2410541394
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11332
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
blueorangutan commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2410390534
@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
DaanHoogland commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2410388623
@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
tpodowd commented on PR #9748:
URL: https://github.com/apache/cloudstack/pull/9748#issuecomment-2406487745
I realised that I should not change the key names that the Object Store
Details use as they may be read/updated outside of the plugin. Thanks!
--
This is an automated message from th
36 matches
Mail list logo
