Request for Recommended Issues to Contribute To
Hi Apache CloudStack Team, We’re two students from the University of Texas at Austin working on a project for our virtualization class, where we are required to contribute to an open source virtualization project. As part of our project requirements, we need to work on a total of three issues. We have already found two with sufficient engagement, but we’re still looking for a third issue that would be suitable for contribution and has some level of maintainer involvement or feedback. If there are any open issues that you think would be a good fit for contributors with general experience in virtualization and cloud infrastructure (but who are new to CloudStack), we’d greatly appreciate any suggestions or guidance you can offer. Thanks in advance for your time and support! Best regards, Jerome Conkright & Brady Wilkin jerome.conkri...@utexas.edu bradywil...@utexas.edu
Re: [PR] Allow users to send requests from CMK using POST requests. [cloudstack-cloudmonkey]
kevin-lii commented on code in PR #161:
URL:
https://github.com/apache/cloudstack-cloudmonkey/pull/161#discussion_r2056705348
##
config/config.go:
##
@@ -282,6 +284,9 @@ func saveConfig(cfg *Config) *Config {
core.AutoComplete = true
core.Output = JSON
}
+ if conf.Section(ini.DEFAULT_SECTION).HasKey("postrequest") {
Review Comment:
Should be fixed!
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Allow users to send requests from CMK using POST requests. [cloudstack-cloudmonkey]
sureshanaparti commented on code in PR #161:
URL:
https://github.com/apache/cloudstack-cloudmonkey/pull/161#discussion_r2055888026
##
config/config.go:
##
@@ -282,6 +284,9 @@ func saveConfig(cfg *Config) *Config {
core.AutoComplete = true
core.Output = JSON
}
+ if conf.Section(ini.DEFAULT_SECTION).HasKey("postrequest") {
Review Comment:
```suggestion
if !conf.Section(ini.DEFAULT_SECTION).HasKey("postrequest") {
core.PostRequest = false
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056053999
##
cloudstack/AccountService.go:
##
@@ -1104,7 +1104,7 @@ func (s *AccountService)
NewIsAccountAllowedToCreateOfferingsWithTagsParams() *I
// Return true if the specified account is allowed to create offerings with
tags.
func (s *AccountService) IsAccountAllowedToCreateOfferingsWithTags(p
*IsAccountAllowedToCreateOfferingsWithTagsParams)
(*IsAccountAllowedToCreateOfferingsWithTagsResponse, error) {
- resp, err :=
s.cs.newRequest("isAccountAllowedToCreateOfferingsWithTags", p.toURLValues())
+ resp, err :=
s.cs.newPostRequest("isAccountAllowedToCreateOfferingsWithTags",
p.toURLValues())
Review Comment:
```suggestion
resp, err :=
s.cs.newRequest("isAccountAllowedToCreateOfferingsWithTags", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Changing executeAPI to utilize POST request [cloudstack-csbench]
sureshanaparti commented on code in PR #18:
URL: https://github.com/apache/cloudstack-csbench/pull/18#discussion_r2055881484
##
apirunner/apirunner.go:
##
@@ -124,11 +125,12 @@ func executeAPIandCalculate(profileName string, apiURL
string, command string, p
var avgTime float64
var totalTime float64
var count float64
+ isGetRequest, _ := regexp.MatchString("^(get|list|query|find)(\\w+)+$",
command)
Review Comment:
```suggestion
isGetRequest, _ :=
regexp.MatchString("^(get|list|query|find|ready)(\\w+)+$", command)
```
for readyForShutdown call
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056077237
##
cloudstack/CloudianService.go:
##
@@ -51,7 +51,7 @@ func (s *CloudianService) NewCloudianIsEnabledParams()
*CloudianIsEnabledParams
// Checks if the Cloudian Connector is enabled
func (s *CloudianService) CloudianIsEnabled(p *CloudianIsEnabledParams)
(*CloudianIsEnabledResponse, error) {
- resp, err := s.cs.newRequest("cloudianIsEnabled", p.toURLValues())
+ resp, err := s.cs.newPostRequest("cloudianIsEnabled", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("cloudianIsEnabled", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056084151
##
cloudstack/ManagementService.go:
##
@@ -804,7 +804,7 @@ func (s *ManagementService) NewReadyForShutdownParams()
*ReadyForShutdownParams
// Returns the status of CloudStack, whether a shutdown has been triggered and
if ready to shutdown
func (s *ManagementService) ReadyForShutdown(p *ReadyForShutdownParams)
(*ReadyForShutdownResponse, error) {
- resp, err := s.cs.newRequest("readyForShutdown", p.toURLValues())
+ resp, err := s.cs.newPostRequest("readyForShutdown", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("readyForShutdown", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Changing executeAPI to utilize POST request [cloudstack-csbench]
sureshanaparti commented on code in PR #18:
URL: https://github.com/apache/cloudstack-csbench/pull/18#discussion_r2055881484
##
apirunner/apirunner.go:
##
@@ -124,11 +125,12 @@ func executeAPIandCalculate(profileName string, apiURL
string, command string, p
var avgTime float64
var totalTime float64
var count float64
+ isGetRequest, _ := regexp.MatchString("^(get|list|query|find)(\\w+)+$",
command)
Review Comment:
```suggestion
isGetRequest, _ :=
regexp.MatchString("^(get|list|query|find|ready|isAccount|cloudianIs)(\\w+)+$",
command)
```
for _readyForShutdown_, _isAccountAllowedToCreateOfferingsWithTags_ and
_cloudianIsEnabled_ calls
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Changing executeAPI to utilize POST request [cloudstack-csbench]
sureshanaparti commented on code in PR #18:
URL: https://github.com/apache/cloudstack-csbench/pull/18#discussion_r2055881484
##
apirunner/apirunner.go:
##
@@ -124,11 +125,12 @@ func executeAPIandCalculate(profileName string, apiURL
string, command string, p
var avgTime float64
var totalTime float64
var count float64
+ isGetRequest, _ := regexp.MatchString("^(get|list|query|find)(\\w+)+$",
command)
Review Comment:
```suggestion
isGetRequest, _ :=
regexp.MatchString("^(get|list|query|find|ready|isAccount|cloudianIs)(\\w+)+$",
command)
```
for readyForShutdown, isAccountAllowedToCreateOfferingsWithTags and
cloudianIsEnabled calls
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056111744
##
cloudstack/QuotaService.go:
##
@@ -408,7 +408,7 @@ func (s *QuotaService) NewQuotaIsEnabledParams()
*QuotaIsEnabledParams {
// Return true if the plugin is enabled
func (s *QuotaService) QuotaIsEnabled(p *QuotaIsEnabledParams)
(*QuotaIsEnabledResponse, error) {
- resp, err := s.cs.newRequest("quotaIsEnabled", p.toURLValues())
+ resp, err := s.cs.newPostRequest("quotaIsEnabled", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("quotaIsEnabled", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056112815
##
cloudstack/QuotaService.go:
##
@@ -598,7 +598,7 @@ func (s *QuotaService) NewQuotaStatementParams(account
string, domainid string,
// Create a quota statement
func (s *QuotaService) QuotaStatement(p *QuotaStatementParams)
(*QuotaStatementResponse, error) {
- resp, err := s.cs.newRequest("quotaStatement", p.toURLValues())
+ resp, err := s.cs.newPostRequest("quotaStatement", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("quotaStatement", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056115362
##
cloudstack/QuotaService.go:
##
@@ -1449,7 +1449,7 @@ func (s *QuotaService) NewQuotaTariffListParams()
*QuotaTariffListParams {
// Lists all quota tariff plans
func (s *QuotaService) QuotaTariffList(p *QuotaTariffListParams)
(*QuotaTariffListResponse, error) {
- resp, err := s.cs.newRequest("quotaTariffList", p.toURLValues())
+ resp, err := s.cs.newPostRequest("quotaTariffList", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("quotaTariffList", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056114303
##
cloudstack/QuotaService.go:
##
@@ -801,7 +801,7 @@ func (s *QuotaService) NewQuotaSummaryParams()
*QuotaSummaryParams {
// Lists balance and quota usage for all accounts
func (s *QuotaService) QuotaSummary(p *QuotaSummaryParams)
(*QuotaSummaryResponse, error) {
- resp, err := s.cs.newRequest("quotaSummary", p.toURLValues())
+ resp, err := s.cs.newPostRequest("quotaSummary", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("quotaSummary", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Modify State Changing Requests to Utilize POST Request [cloudstack-go]
sureshanaparti commented on code in PR #107:
URL: https://github.com/apache/cloudstack-go/pull/107#discussion_r2056117130
##
cloudstack/QuotaService.go:
##
@@ -192,7 +192,7 @@ func (s *QuotaService) NewQuotaBalanceParams(account
string, domainid string) *Q
// Create a quota balance statement
func (s *QuotaService) QuotaBalance(p *QuotaBalanceParams)
(*QuotaBalanceResponse, error) {
- resp, err := s.cs.newRequest("quotaBalance", p.toURLValues())
+ resp, err := s.cs.newPostRequest("quotaBalance", p.toURLValues())
Review Comment:
```suggestion
resp, err := s.cs.newRequest("quotaBalance", p.toURLValues())
```
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Re: [PR] Changing executeAPI to utilize POST request [cloudstack-csbench]
sureshanaparti commented on code in PR #18:
URL: https://github.com/apache/cloudstack-csbench/pull/18#discussion_r2056132609
##
apirunner/apirunner.go:
##
@@ -124,11 +125,12 @@ func executeAPIandCalculate(profileName string, apiURL
string, command string, p
var avgTime float64
var totalTime float64
var count float64
+ isGetRequest, _ := regexp.MatchString("^(get|list|query|find)(\\w+)+$",
command)
Review Comment:
also, this quota calls - _quotaIsEnabled, quotaTariffList, quotaSummary,
quotaBalance, quotaStatement_ can be added to GET list.
--
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: dev-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Request for Recommended Issues to Contribute To
Hi Apache CloudStack Team, We’re two students from the University of Texas at Austin working on a project for our virtualization class, where we are required to contribute to an open source virtualization project. As part of our project requirements, we need to work on a total of three issues. We have already found two with sufficient engagement, but we’re still looking for a third issue that would be suitable for contribution and has some level of maintainer involvement or feedback. If there are any open issues that you think would be a good fit for contributors with general experience in virtualization and cloud infrastructure (but who are new to CloudStack), we’d greatly appreciate any suggestions or guidance you can offer. Thanks in advance for your time and support! Best regards, Jerome Conkright & Brady Wilkin jerome.conkri...@utexas.edu bradywil...@utexas.edu
Re: [PR] Documentation related to Instance lease feature [cloudstack-documentation]
sudo87 commented on PR #492: URL: https://github.com/apache/cloudstack-documentation/pull/492#issuecomment-2826302827 @blueorangutan docbuild -- 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: dev-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Documentation related to Instance lease feature [cloudstack-documentation]
blueorangutan commented on PR #492: URL: https://github.com/apache/cloudstack-documentation/pull/492#issuecomment-2826306235 QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/492. (QA-JID 324) -- 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: dev-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Documentation related to Instance lease feature [cloudstack-documentation]
blueorangutan commented on PR #492: URL: https://github.com/apache/cloudstack-documentation/pull/492#issuecomment-2826303788 @sudo87 a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. -- 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: dev-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: Request for Recommended Issues to Contribute To
Hi Jerome and Brady, You may find this CloudStack self-learning course useful: https://github.com/shapeblue/hackerbook along with https://github.com/shapeblue/mbx to create local CloudStack dev-testing environments on Ubuntu/Fedora/EL. You can explore issues from here https://github.com/apache/cloudstack/issues but also: https://github.com/apache/cloudstack-cloudmonkey/issues Https://github.com/apache/cloudstack-terraform-provider/issues https://github.com/apache/cloudstack-kubernetes-provider/issues Do you have a particular preference for any type of issues you'd like to work on, such as on the UI, build system, business logic, hypervisor (kvm, vmware, xs/xcp-ng), storage, network etc? Regards. From: Jerome B Conkright Sent: Wednesday, April 23, 2025 23:17 To: dev@cloudstack.apache.org Subject: Request for Recommended Issues to Contribute To Hi Apache CloudStack Team, We’re two students from the University of Texas at Austin working on a project for our virtualization class, where we are required to contribute to an open source virtualization project. As part of our project requirements, we need to work on a total of three issues. We have already found two with sufficient engagement, but we’re still looking for a third issue that would be suitable for contribution and has some level of maintainer involvement or feedback. If there are any open issues that you think would be a good fit for contributors with general experience in virtualization and cloud infrastructure (but who are new to CloudStack), we’d greatly appreciate any suggestions or guidance you can offer. Thanks in advance for your time and support! Best regards, Jerome Conkright & Brady Wilkin jerome.conkri...@utexas.edu bradywil...@utexas.edu
