Re: [I] v2.15.0 breaking change regarding displayText handling for CreateNetwork [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on issue #70: URL: https://github.com/apache/cloudstack-go/issues/70#issuecomment-1935474971 Since this is optional in newer versions but not removed, the go-sdk can still call/pass it? -- This is an automated message from the Apache Git Service. To respond to th

Re: [I] Allow for CTRL-C to break the current operation [cloudstack-cloudmonkey]

2024-02-08 Thread via GitHub
rohityadavcloud commented on issue #126: URL: https://github.com/apache/cloudstack-cloudmonkey/issues/126#issuecomment-1935448872 Hi @serverchief I've added some changes to allow breaking an long-running async job polling, for example: ``` (homecloud) šŸµ > deploy virtualmachine servic

Re: [I] v2.15.0 breaking change regarding displayText handling for CreateNetwork [cloudstack-go]

2024-02-08 Thread via GitHub
shwstppr commented on issue #70: URL: https://github.com/apache/cloudstack-go/issues/70#issuecomment-1935401821 @rohityadavcloud @hrak what should be the way forward here? With each ACS release, there will be some breaking changes so should we try to maintain backward compatibility or shoul

Re: [I] bug: async client don't support timeout [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on issue #36: URL: https://github.com/apache/cloudstack-go/issues/36#issuecomment-1935385888 Closing this on @tetra12 's remarks at https://github.com/apache/cloudstack-go/pull/37#issuecomment-1779378444 `This refactor is NOT required, since current API implement

Re: [I] bug: async client don't support timeout [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud closed issue #36: bug: async client don't support timeout URL: https://github.com/apache/cloudstack-go/issues/36 -- 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.

Re: [I] quota api calls are missing [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on issue #54: URL: https://github.com/apache/cloudstack-go/issues/54#issuecomment-1935384923 Currently also missing: ``` go run generate/generate.go generate/layout.go --api=generate/listApis.json 2024/02/09 06:04:38 Api missing in layout: createTungstenFabr

Re: [I] Missing API support [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on issue #22: URL: https://github.com/apache/cloudstack-go/issues/22#issuecomment-1935383928 Currently with 4.19/main: ``` go run generate/generate.go generate/layout.go --api=generate/listApis.json 2024/02/09 06:04:38 Api missing in layout: createTungstenFa

Re: [PR] Bump golang.org/x/net from 0.8.0 to 0.17.0 [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud merged PR #72: URL: https://github.com/apache/cloudstack-go/pull/72 -- 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...@clou

Re: [PR] Fix spelling [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud merged PR #74: URL: https://github.com/apache/cloudstack-go/pull/74 -- 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...@clou

Re: [PR] Bump golang.org/x/net from 0.8.0 to 0.17.0 [cloudstack-go]

2024-02-08 Thread via GitHub
dependabot[bot] commented on PR #72: URL: https://github.com/apache/cloudstack-go/pull/72#issuecomment-1935381478 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, l

Re: [PR] Bump golang.org/x/net from 0.8.0 to 0.17.0 [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud closed pull request #72: Bump golang.org/x/net from 0.8.0 to 0.17.0 URL: https://github.com/apache/cloudstack-go/pull/72 -- 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

Re: [PR] 4.19 support changes [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on PR #76: URL: https://github.com/apache/cloudstack-go/pull/76#issuecomment-1935381113 LGTM - merged @shwstppr -- 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 spec

Re: [PR] 4.19 support changes [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud merged PR #76: URL: https://github.com/apache/cloudstack-go/pull/76 -- 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...@clou

Re: [PR] 4.19 support changes [cloudstack-go]

2024-02-08 Thread via GitHub
shwstppr commented on PR #76: URL: https://github.com/apache/cloudstack-go/pull/76#issuecomment-1935378963 Tests are passing both locally and in GH action -- 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 t

Re: [PROPOSE] ACS 4.18.2.0

2024-02-08 Thread JoĆ£o Jandre Paraquetti
Hi, Rohit, all I don't see any critical problems that should be addressed ASAP on the 4.18 branch; therefore, there should be no problem with postponing 4.18.2 a little bit. We still have several issues targeted at 4.18.2 that have yet to be addressed, so a couple more weeks before the freeze

Re: Discussion: CloudStack upgrade to JRE17 and Debian12/python3

2024-02-08 Thread Rohit Yadav
Thanks Wei. I think VR should be set to 512 (if memory consuming is above 300MB). SSVM and CPVM arenā€™t they already at 512-1024 MiB? Regards. From: Nux Sent: Thursday, February 8, 2024 2:50:11 PM To: dev@cloudstack.apache.org Cc: Wei ZHOU Subject: Re: Disc

Re: Regarding the Log4j upgrade

2024-02-08 Thread Guto Veronezi
Hello guys We finally merged PR #7131 [1]. With that, other PRs targeted to the branch "main" might get the conflict status. The PR #7131 [1] description contains instructions on how to fix the conflicts; however, if you have any doubts, do not hesitate to contact us. For those who have PRs t

Re: [PR] 4.19 support changes [cloudstack-go]

2024-02-08 Thread via GitHub
rohityadavcloud commented on PR #76: URL: https://github.com/apache/cloudstack-go/pull/76#issuecomment-1934036466 LGTM - didn't test it. Are there basic build/tests that cover basic checks? -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[DISCUSS][PROPOSAL] web site publishing procedure

2024-02-08 Thread Daan Hoogland
devs (and PMC), I created a procedure based on a staging->production promotion of PRs. it is described in the readme file [1] I went ahead and created it as we didn't have anything before. That does not mean I get to decide on my own, so please add your unsalted comment or create/add change propo

Re: [PR] 4.19 support changes [cloudstack-go]

2024-02-08 Thread via GitHub
shwstppr commented on PR #76: URL: https://github.com/apache/cloudstack-go/pull/76#issuecomment-1933949015 @Pearl1594 @rohityadavcloud @vishesh92 If you approve, we can merge and create a new release -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: Discussion: CloudStack upgrade to JRE17 and Debian12/python3

2024-02-08 Thread Nux
+1 option 2 (Debian12). I think we need to do a bit longer term testing before we decide on the new RAM size for the VR. 384MB might be good to boot, but the slightest memory leak or a temporarily spiky conntrack table could derail it. ..But yeah, need to be conscious of the overall memory over