Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-06-18 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-2175723629 > @weizhouapache is this ok by you? or else what still needs to be done ? cc @Pearl1594 ? @DaanHoogland @Pearl1594 verified ok - create a user data with "hah

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-06-18 Thread via GitHub
weizhouapache merged PR #8791: URL: https://github.com/apache/cloudstack/pull/8791 -- 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: commits-unsubscr...@c

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-06-18 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-2175724016 thanks @Pearl1594 , merging -- 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 sp

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-06-18 Thread via GitHub
DaanHoogland commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-2175531867 @weizhouapache is this ok by you? or else what still needs to be done ? cc @Pearl1594 ? -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-05-30 Thread via GitHub
Pearl1594 commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1620460659 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = function (

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-05-30 Thread via GitHub
shwstppr commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1620255651 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = function (t

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-15 Thread via GitHub
Pearl1594 commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1526368044 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = function (

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-15 Thread via GitHub
weizhouapache commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1526235912 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = functi

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-15 Thread via GitHub
weizhouapache commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1526218947 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = functi

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-15 Thread via GitHub
Pearl1594 commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1526190752 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = function (

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-15 Thread via GitHub
weizhouapache commented on code in PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#discussion_r1526044811 ## ui/src/utils/plugins.js: ## @@ -490,8 +499,7 @@ export const genericUtilPlugin = { } app.config.globalProperties.$toBase64AndURIEncoded = functi

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1998069746 > Ok got it, I'll add a checkbox that will only be used by the ui to determine whether to encode the provided userdata or not.. good ,looking forward to it -- This is an

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
Pearl1594 commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1998061391 Ok got it, I'll add a checkbox that will only be used by the ui to determine whether to encode the provided userdata or not.. -- This is an automated message from the Apache Git S

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1998055575 > @weizhouapache sorry I didn't quite understand why the gzip in between. refer to https://cloudinit.readthedocs.io/en/latest/explanation/format.html#gzip-compressed-conte

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
Pearl1594 commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1997986975 @weizhouapache sorry I didn't quite understand why the gzip in between. Because without it i.e., `echo "Apache CloudStack" | base64 -w0` if this is provided as input to the userdat

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1997710164 > @weizhouapache we could - do you mean that users would need to specify if its encoded or not? If so, wouldn't it be user friendly if we have the code decide if it's encoded or

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
Pearl1594 commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1997692496 @weizhouapache we could - do you mean that users would need to specify if its encoded or not? If so, wouldn't it be user friendly if we have the code decide if it's encoded or not an

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
weizhouapache commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1997682818 @Pearl1594 what do you think if we add a checkbox to indicate if userdata is base64-encoded ? for example ![image](https://github.com/apache/cloudstack/assets/57355

Re: [PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
codecov[bot] commented on PR #8791: URL: https://github.com/apache/cloudstack/pull/8791#issuecomment-1997542522 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8791?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

[PR] UI: Fix Userdata registration from UI [cloudstack]

2024-03-14 Thread via GitHub
Pearl1594 opened a new pull request, #8791: URL: https://github.com/apache/cloudstack/pull/8791 ### Description This PR fixes: https://github.com/apache/cloudstack/issues/8736 ### Types of changes - [ ] Breaking change (