-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
service and disk offerings #77
Conversation
Hi @rohityadavcloud, any tentative timelines when v0.5.0 would release? Already merged features could be of a lot of use 😄 |
Hi @adidiborg welcome to the opensource project, I hear you and I hope the community can work towards a release soon. I hope the next TF release will be sometime in coming months. Anybody in the community can work to become a committer and PMC member, and lead release work who work in their "free" time, and it's highly encouraged that users and organisations participate and sponsor their team to work in the community. It's often seen many interested individuals and organisations come together to work on their features and sometimes releases, and often some organisations sponsor feature and maintenance work, and collaborate for testing and release work but not just limited to coding but also documentation, website, marketing and events. Everyone is welcome to start discussions on our dev@ and users@ mailing lists, in fact your comment would be more visible on the lists unless it was only specifically for me. |
thanks for responding, I will also try to contribute whereever I can |
Update - I'd still like to get these merged once the cloudstack SDK is updated. |
@poddm can you address conflicts on your PR? |
@fabiomatavelli @kiranchavala @poddm Can we come to an agreement on service_offerings between this PR and what @fabiomatavelli had worked on here #113 and move forward with one of these solutions? This is starting to become an issue for me to be able to manage service offerings for our customers and this would be a huge help to be able to create less offerings because of being able to have offerings that are more dynamic to use. P.S. @kiranchavala I would like to see a new release this quarter (end of Aug would be nice 😄 ) What can we do to help move the release of a version? cc: @rohityadavcloud |
I can remove I'd like to keep the separate types to align the GUI presentation and make it easier to understand what fields are required.
|
@poddm @fabiomatavelli I was on vacation so couldn't reply earlier. Longer term, since you're both active contributors and if you want to main and drive releases yourselves I can propose you both as committers after which you can propose yourself to be release managers (did some write up here on the typical release process: https://cwiki.apache.org/confluence/display/CLOUDSTACK/%5BDRAFT%5D+How+to+Become+a+CloudStack+Release+Manager ) - let me know if you're interested to do that. Looking at $dayjob pile and on-going community work, I think most of us will busy this/next month working towards the upcoming ACS 4.20 and other misc releases. We'll be available most likely in end of Q3 or in Q4 for the next ACS Terraform (Tofu) Provider releases. Meanwhile, feel free to work on stabilising the PRs & overall main branch to be ready for release work and continue to build/use the snapshots as required. I've pinged @kiranchavala @vishesh92 who can assist with review related queries. |
@rohityadavcloud, I'm interested. |
@poddm The build is failing. Please check. |
@vishesh92 I don't believe the comment about "removing existing resource types" is what you think. I understood it to mean he would remove his changes in this PR that are not related to the "service offerings" resource. Also, @poddm if we are moving forward with your approach, can we use the new terraform plugin framework approach? #113 I think we should start moving to using that for new/updated resources. Then eventually we can get to upgrading the rest to use the new framework. |
@poddm @vishesh92 @rohityadavcloud Is there anything I can help with to get this merged? I'm desperately needing to have this avail, so I can pass this on to our customers and help reduce a lot of manual management of service offerings. |
I missed these previous messages. Let me look at refactoring this into the new framework SDK. |
@poddm Any update in this? I really would like to have the dynamic offerings soon. This would be a HUGE benefit for our customers right now. Hopefully we can get the next release out soon too 😄 |
@poddm @rohityadavcloud Since we are now in the 4th quarter, I was hoping we can start wrapping things like this up and get a new release out soon. This one PR in particular is a significant one for me. I really don't want to fork this and start down a different path, but at some point I might have to make that decision. I know everyone is busy and completely appreciate all the hard work that goes into this project. I'm hoping we can get this long lived PR over the line soon and also get a release out soon. |
@poddm Unless I'm missing something, shouldn't there be updates to the cloudstack_instance resource so when using a constrained/dynamic offering we can specify the number of CPU, Memory...etc? |
Hi @CodeBleu on the next release, we can try to start some release work in end Nov & Dec, I don't think some of us will have bandwidth until CCC (Nov 22-23). I've started this issue to track & discuss the next release, we also have a Github release milestone; #137 Usually I'm up for merging this PR but we need the merge criteria (review/update as required) to pass, for example - the Github actions acceptance tests should pass, 1-2 code review/lgtms & manual QA. Can you and/or @poddm and others help advise on that? |
@poddm can you review/rebase the PR & look at failing tests. Thanks. |
@poddm I see you're working on this. Thanks!! I've been trying to test things before your recent changes, and have pulled down your recent changes too, but I believe the code is still missing the option to specify a |
This PR has been replaced with this PR |
Enhanced the following resources to support the full api spec.
I split the service offering API up into multiple resources to make it a bit clearer of the different qos types