Parameterized content_cli for Wind River#2522
Parameterized content_cli for Wind River#2522oloten wants to merge 3 commits intocloudify-cosmo:masterfrom
Conversation
Applied variables and updated titles for product names in the files under content_cli
| {{< highlight bash >}} | ||
|
|
||
| $ cfy cluster status | ||
| Retrieving Cloudify cluster status... [ip=10.239.1.160] |
There was a problem hiding this comment.
We can't remove it as it's the command line output. I'll suggest parametrizing it, if possible, otherwise we need to keep it as the change requires code changes in our CLI.
There was a problem hiding this comment.
Updated by adding "Cloudify" back to text.
content/cli/maint_cli/license.md
Outdated
| Uploading license `license.yaml` to the Manager... | ||
| License successfully uploaded. |
There was a problem hiding this comment.
We can't remove it as it's the command line output. I'll suggest parametrizing it, if possible, otherwise we need to keep it as the change requires code changes in our CLI.
There was a problem hiding this comment.
Cloudify text restored.
content/cli/maint_cli/license.md
Outdated
| Retrieving License | ||
| ... | ||
| Cloudify License | ||
| License |
There was a problem hiding this comment.
We can't remove it as it's the command line output. I'll suggest parametrizing it, if possible, otherwise we need to keep it as the change requires code changes in our CLI.
content/cli/maint_cli/license.md
Outdated
| Removing License from the Manager... | ||
| License successfully removed. |
There was a problem hiding this comment.
We can't remove it as it's the command line output. I'll suggest parametrizing it, if possible, otherwise we need to keep it as the change requires code changes in our CLI.
There was a problem hiding this comment.
Restored - Until the CLI is updated, we should align the text with whatever the users sees.
content/cli/maint_cli/profiles.md
Outdated
| --- | ||
| title: profiles | ||
| description: The `cfy profiles` command is used to manage the Cloudify profiles. | ||
| description: The `cfy profiles` command is used to manage product profiles. |
There was a problem hiding this comment.
| description: The `cfy profiles` command is used to manage product profiles. | |
| description: The `cfy profiles` command is used to manage the product profiles. |
There was a problem hiding this comment.
Fixed per suggestion
content/cli/orch_cli/_index.md
Outdated
| {{% /note %}} | ||
|
|
||
| If you haven't already [installed Cloudify]({{< relref "cloudify_manager/cloudify_cli" >}}), now would be a good time to do so. | ||
| If you haven't already [{{< param product_name >}}]({{< relref "cloudify_manager/cloudify_cli" >}}), now would be a good time to do so. |
There was a problem hiding this comment.
| If you haven't already [{{< param product_name >}}]({{< relref "cloudify_manager/cloudify_cli" >}}), now would be a good time to do so. | |
| If you haven't already [installed {{< param product_name >}}]({{< relref "cloudify_manager/cloudify_cli" >}}), now would be a good time to do so. |
There was a problem hiding this comment.
Fixed in next commit
content/cli/orch_cli/_index.md
Outdated
| # Configuration | ||
|
|
||
| By default, a `.cloudify` directory is created under ~(Home directory). You can change the location using an `_env_` variable (for example, `cfy init` or `cfy profiles use`). | ||
| By default, a `.{{< param product_name >}}` directory is created under ~(Home directory). You can change the location using an `_env_` variable (for example, `cfy init` or `cfy profiles use`). |
There was a problem hiding this comment.
the parameter product_name is Cloudify with capital C the directory is with lower c and it's case sensitive
There was a problem hiding this comment.
Fixed in next commit
content/cli/orch_cli/_index.md
Outdated
| {{% note title="Note" %}} | ||
|
|
||
| The `.cloudify` directory might also contain other files and directories, which are used internally by `cfy`, that are not described in this section. | ||
| The `.{{< param product_name >}}` directory might also contain other files and directories, which are used internally by `cfy`, that are not described in this section. |
There was a problem hiding this comment.
the parameter product_name is Cloudify with capital C the directory is with lower c and it's case sensitive
There was a problem hiding this comment.
Fixed in next commit
|
OK - thanks for the review Alex - I will make the changes and post an updated commit |
Revised with comments from initial review. Restored "Cloudify" test where required.
|
Hello Alex, |
2 participants
Applied variables and updated titles for product names in the files under content_cli. This is create more generic content