[GitHub] [cloudstack-terraform-provider] harikrishna-patnala commented on a change in pull request #6: Fix Makefile and handle error checks

2021-10-06 Thread GitBox
harikrishna-patnala commented on a change in pull request #6: URL: https://github.com/apache/cloudstack-terraform-provider/pull/6#discussion_r723877698 ## File path: GNUmakefile ## @@ -29,10 +29,10 @@ fmt: gofmt -w $(GOFMT_FILES) fmtcheck: - @sh -c "'$(CURDIR)

[GitHub] [cloudstack-cloudmonkey] rhtyd commented on issue #105: How to resolve delete trafficmonitor Error message

2021-10-06 Thread GitBox
rhtyd commented on issue #105: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/105#issuecomment-937477143 @cdaehyun0 I think this is possibly an issue with CloudStack (if at all) and not a bug in cmk. Do you only have a single host? Usually, you would put the host in maintena

[GitHub] [cloudstack-cloudmonkey] cdaehyun0 commented on issue #105: How to resolve delete trafficmonitor Error message

2021-10-06 Thread GitBox
cdaehyun0 commented on issue #105: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/105#issuecomment-937475072 > > > @cdaehyun0, you have to put the host into maintenance mode. The message, in your case, is misleading because there isn't an API param `force` in the `de

[GitHub] [cloudstack-cloudmonkey] slavkap edited a comment on issue #105: How to resolve delete trafficmonitor Error message

2021-10-06 Thread GitBox
slavkap edited a comment on issue #105: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/105#issuecomment-937461432 @cdaehyun0, you have to put the host into maintenance mode. The message, in your case, is misleading because there isn't an API param `force` in the `deleteTraff

[GitHub] [cloudstack-cloudmonkey] slavkap commented on issue #105: How to resolve delete trafficmonitor Error message

2021-10-06 Thread GitBox
slavkap commented on issue #105: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/105#issuecomment-937461432 @cdaehyun0, you have to put the host into maintenance mode. The message, in your case, is misleading because there isn't an API param `force` in the `deleteTrafficMonit

[GitHub] [cloudstack-cloudmonkey] cdaehyun0 opened a new issue #105: How to resolve delete trafficmonitor Error message

2021-10-06 Thread GitBox
cdaehyun0 opened a new issue #105: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/105 Error Message::: (localcloud) 🐱 > delete trafficmonitor id=2f3710d4-04af-412b-a64a-1aca0cb07c58 🙈 Error: (HTTP 530, error code 4250) Host 2f3710d4-04af-412b-a64a-1aca0cb07c58 c

[GitHub] [cloudstack-documentation] Hudratronium opened a new pull request #247: Additional information and restructurizing

2021-10-06 Thread GitBox
Hudratronium opened a new pull request #247: URL: https://github.com/apache/cloudstack-documentation/pull/247 HI, here my take on the ACL implementation in a VPC context. I added some more information which might be interesting for planning of VPC / tier design as well as security concider

Re: KVM Agent disconnect hooks

2021-10-06 Thread Marcus
Thanks, Daan, I appreciate the feedback and I hope it is useful. I wish I could comment more on the other hypervisors and how something similar might work for them. On Wed, Oct 6, 2021 at 2:49 AM Daan Hoogland wrote: > thanks Marcus, > > On Tue, Oct 5, 2021 at 7:32 PM Marcus wrote: > > > Hi ev

Re: 4.16.0.0 release

2021-10-06 Thread Nicolas Vazquez
Hi All, These are the last outstanding items on the 4.16.0 milestone: https://github.com/apache/cloudstack/milestone/16. I would like to propose/announce a code freeze date for October 10th. After this time, we are starting the stabilisation phase and no more features and general fixes will be

[GitHub] [cloudstack-go] rhtyd commented on pull request #11: Tests for resource operations and services

2021-10-06 Thread GitBox
rhtyd commented on pull request #11: URL: https://github.com/apache/cloudstack-go/pull/11#issuecomment-936135819 @davidjumani are you LGTM, Pearl can iterate on this in another PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [cloudstack-go] rhtyd commented on pull request #13: Adding WithListall option

2021-10-06 Thread GitBox
rhtyd commented on pull request #13: URL: https://github.com/apache/cloudstack-go/pull/13#issuecomment-936135023 @Pearl1594 pl review/test and merge. -- 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

[GitHub] [cloudstack-cloudmonkey] PaulAngus commented on issue #104: Enable passing of profile information in cmk commandline

2021-10-06 Thread GitBox
PaulAngus commented on issue #104: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/104#issuecomment-936043774 Hi Rohit. I want avoid a text file with the endpoint and credentials in it. Using the existing nomenclature there would be -s secret key -k

[GitHub] [cloudstack-cloudmonkey] rhtyd edited a comment on issue #104: Enable passing of profile information in cmk commandline

2021-10-06 Thread GitBox
rhtyd edited a comment on issue #104: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/104#issuecomment-935818274 With latest v6.2.0 you can now pass the -p and -c options to specify custom config/server profile https://github.com/apache/cloudstack-cloudmonkey/wiki/Getting-Sta

[GitHub] [cloudstack-cloudmonkey] rhtyd commented on issue #104: Enable passing of profile information in cmk commandline

2021-10-06 Thread GitBox
rhtyd commented on issue #104: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/104#issuecomment-935818274 With latest v6.2.0 you can now pass the -c option to specify custom config/server profile https://github.com/apache/cloudstack-cloudmonkey/wiki/Getting-Started#getting-he

[GitHub] [cloudstack-cloudmonkey] PaulAngus opened a new issue #104: Enable passing of profile information in cmk commandline

2021-10-06 Thread GitBox
PaulAngus opened a new issue #104: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/104 ### Feature Request **Enable the passing of profile information in cmk command line** ### Use Case - Storing API and Secret Keys in a plain text file presents a large secu

Re: KVM Agent disconnect hooks

2021-10-06 Thread Daan Hoogland
thanks Marcus, On Tue, Oct 5, 2021 at 7:32 PM Marcus wrote: > Hi everyone! It's been awhile. I've got a feature I'd love to get some > feedback on and contribute to the community, if it's acceptable. I need to > brush up on the proper process (did read CONTRIBUTING.md). Not a lot has changed.

[GitHub] [cloudstack-go] davidjumani edited a comment on pull request #11: Tests for resource operations and services

2021-10-06 Thread GitBox
davidjumani edited a comment on pull request #11: URL: https://github.com/apache/cloudstack-go/pull/11#issuecomment-935710321 Would be much better if all the json is in separate testdata json files rather than just mixing it in with the unit test files -- This is an automated message

[GitHub] [cloudstack-go] davidjumani commented on pull request #11: Tests for resource operations and services

2021-10-06 Thread GitBox
davidjumani commented on pull request #11: URL: https://github.com/apache/cloudstack-go/pull/11#issuecomment-935710321 Would be much better if all the json is in separate testdata json files -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [cloudstack-go] davidjumani opened a new pull request #13: Adding WithListall option

2021-10-06 Thread GitBox
davidjumani opened a new pull request #13: URL: https://github.com/apache/cloudstack-go/pull/13 Adds WithListall() to make life easier while fetching by id / name across projects and accounts -- This is an automated message from the Apache Git Service. To respond to the message, please l