[GitHub] [cloudstack-terraform-provider] harikrishna-patnala commented on issue #21: Can't create port forwarding rule

2021-11-29 Thread GitBox
harikrishna-patnala commented on issue #21: URL: https://github.com/apache/cloudstack-terraform-provider/issues/21#issuecomment-981411662 @synergiator on the first try of creating port forwarding rule, I did not find any issue it got successfully created. Then I tried few more scena

[GitHub] [cloudstack-go] davidjumani opened a new pull request #23: Support 4.x compatibility

2021-11-29 Thread GitBox
davidjumani opened a new pull request #23: URL: https://github.com/apache/cloudstack-go/pull/23 Modded version of https://github.com/apache/cloudstack-go/pull/21 Allows backward compatibility of Managementserverid of both int64 and string data types -- This is an automated message

[GitHub] [cloudstack-go] davidjumani commented on pull request #21: handle type compatibilty for 4.x baseline

2021-11-29 Thread GitBox
davidjumani commented on pull request #21: URL: https://github.com/apache/cloudstack-go/pull/21#issuecomment-981429451 @psycofdj I'm unable to edit this PR, so created another one with these commits along with the type name change from CSLong to ManagementServerId in #23 -- This is an

[GitHub] [cloudstack-go] rhtyd commented on pull request #23: Support 4.x compatibility

2021-11-29 Thread GitBox
rhtyd commented on pull request #23: URL: https://github.com/apache/cloudstack-go/pull/23#issuecomment-981430623 @davidjumani how have we tested this? (apart from the unit test) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [cloudstack-go] davidjumani commented on pull request #23: Support 4.x compatibility

2021-11-29 Thread GitBox
davidjumani commented on pull request #23: URL: https://github.com/apache/cloudstack-go/pull/23#issuecomment-981436559 @rhtyd Yes, ran a couple of tests targetting the ManagementServerId with both int and string data types in the response JSON -- This is an automated message from the Apa

[GitHub] [cloudstack-go] psycofdj commented on pull request #21: handle type compatibilty for 4.x baseline

2021-11-29 Thread GitBox
psycofdj commented on pull request #21: URL: https://github.com/apache/cloudstack-go/pull/21#issuecomment-981446386 Alright I shall close this PR then. Thanks @davidjumani -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cloudstack-go] psycofdj closed pull request #21: handle type compatibilty for 4.x baseline

2021-11-29 Thread GitBox
psycofdj closed pull request #21: URL: https://github.com/apache/cloudstack-go/pull/21 -- 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-unsubsc

[GitHub] [cloudstack-go] davidjumani edited a comment on pull request #21: handle type compatibilty for 4.x baseline

2021-11-29 Thread GitBox
davidjumani edited a comment on pull request #21: URL: https://github.com/apache/cloudstack-go/pull/21#issuecomment-981429451 @psycofdj I'm unable to edit this PR, so created another one with these commits along with the type name change from CSLong to UUID in #23 -- This is an automate

[GitHub] [cloudstack-terraform-provider] harikrishna-patnala commented on issue #20: Understanding implications of the legacy SDK notification

2021-11-29 Thread GitBox
harikrishna-patnala commented on issue #20: URL: https://github.com/apache/cloudstack-terraform-provider/issues/20#issuecomment-98147 @synergiator I'm also not aware of this but on quick search on google, came to know this error comes from terraform core when it finds that legacy SDK

[GitHub] [cloudstack-go] rhtyd commented on pull request #23: Support 4.x compatibility

2021-11-29 Thread GitBox
rhtyd commented on pull request #23: URL: https://github.com/apache/cloudstack-go/pull/23#issuecomment-981493465 Alright @psycofdj @davidjumani thanks for coming to some solution that satisfies both your requirements. -- This is an automated message from the Apache Git Service. To respon

[GitHub] [cloudstack-go] rhtyd merged pull request #23: Support 4.x compatibility

2021-11-29 Thread GitBox
rhtyd merged pull request #23: URL: https://github.com/apache/cloudstack-go/pull/23 -- 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..

[GitHub] [cloudstack-go] rhtyd commented on issue #15: Error retrieving ID of template: json: cannot unmarshal object into Go struct field alias.childtemplates of type []interface {}

2021-11-29 Thread GitBox
rhtyd commented on issue #15: URL: https://github.com/apache/cloudstack-go/issues/15#issuecomment-981494227 Perhaps this is no longer an issue now, can you check and confirm @nishiokay @synergiator ? Thanks. -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [cloudstack-terraform-provider] harikrishna-patnala commented on issue #19: Add renovate bot integration for advanced dependency management

2021-11-29 Thread GitBox
harikrishna-patnala commented on issue #19: URL: https://github.com/apache/cloudstack-terraform-provider/issues/19#issuecomment-981497293 Thanks @synergiator for the suggestion, I'll keep this ticket open for tracking so that we can work on it for next release. Other issue I could s

[DISCUSS] 2FA framework and plugins for CloudStack

2021-11-29 Thread Rohit Yadav
All, During CCC21 hackathon, I explored the feasibility of a 2FA framework and a TOTP (time-based OTP) plugin that can be used with Google Authenticator, MS Authenticator, Authy etc. I've used ideas of TOTP based 2FA PoC to put together a design doc for discussion: https://cwiki.apache.org/con

[GitHub] [cloudstack-go] davidjumani opened a new pull request #24: Adding kubernetes service

2021-11-29 Thread GitBox
davidjumani opened a new pull request #24: URL: https://github.com/apache/cloudstack-go/pull/24 Added as part of https://github.com/apache/cloudstack-terraform-provider/issues/13 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [cloudstack-go] davidjumani commented on issue #15: Error retrieving ID of template: json: cannot unmarshal object into Go struct field alias.childtemplates of type []interface {}

2021-11-29 Thread GitBox
davidjumani commented on issue #15: URL: https://github.com/apache/cloudstack-go/issues/15#issuecomment-981553649 This appears to be an issue with parsing the listTemplate API response, @nishiokay could you provide the response to the listTemplate response with the templateid 450ec62f-16b1

[GitHub] [cloudstack-go] davidjumani edited a comment on issue #15: Error retrieving ID of template: json: cannot unmarshal object into Go struct field alias.childtemplates of type []interface {}

2021-11-29 Thread GitBox
davidjumani edited a comment on issue #15: URL: https://github.com/apache/cloudstack-go/issues/15#issuecomment-981553649 This appears to be an issue with parsing the listTemplate API response, @nishiokay could you : - List the template (which has child templates) via the sdk - Provide

[GitHub] [cloudstack-terraform-provider] harikrishna-patnala commented on issue #18: API backward compatibility problem while deploying virtual machines using human readable template reference name

2021-11-29 Thread GitBox
harikrishna-patnala commented on issue #18: URL: https://github.com/apache/cloudstack-terraform-provider/issues/18#issuecomment-981568225 @synergiator I checked the other ticket on cloudstack-go and you are saying it is not reproducible now. Is it still a problem deploying a VM using terr

[GitHub] [cloudstack-terraform-provider] synergiator commented on issue #18: API backward compatibility problem while deploying virtual machines using human readable template reference name

2021-11-29 Thread GitBox
synergiator commented on issue #18: URL: https://github.com/apache/cloudstack-terraform-provider/issues/18#issuecomment-981628455 This data is from API 4.11 (a patched fork of it though); with the terraform provider I can reproduce this. Will run another unit test for cloudstack-go to con

[DISCUSS] SystemVM Optimizations

2021-11-29 Thread Pearl d'Silva
Hi All, In 4.16 we introduced an enhancement that provided support for auto upgrades, where in one wouldn't have to register the systemVM template prior upgrading their environments. This however led to an increased size of the cloudstack-management package as the systemVM templates for all the