-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MTA-2744 - Rework MTA CLI section #880
base: main
Are you sure you want to change the base?
Changes from 45 commits
016bdd6
7a5011b
94897b6
54a0ec2
3815164
8722f49
4106033
ad4158d
a9adfb0
1b2901f
37f5272
5dae139
74433e1
555a837
b27fa85
9d67697
aed8816
0e6ef1e
ed6cf04
56a8f60
da19c36
896b888
e90fd7d
35307af
30ba650
ad8a8d9
e57201e
a28e029
1474e60
bb03d4e
1504f44
1011b1f
5e6d4b9
c2d899e
b86383b
78c393b
728e9b8
222c1f3
47a99ce
85a35bf
b84f8d0
0765fe5
bfe9536
b825cac
176ec50
7e09487
142dbdf
e6dbf74
a212ed2
5d740d7
97ad2ea
8af6fd0
a850e14
760771b
7bbe1a8
5910a92
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -4,4 +4,9 @@ html/ | |||||
*/build/* | ||||||
Gemfile.lock | ||||||
*/master.html | ||||||
.idea/ | ||||||
.idea/ | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
.gitignore | ||||||
.DS_Store | ||||||
assets/.DS_Store | ||||||
docs/.DS_Store | ||||||
docs/topics/.DS_Store |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,29 +16,26 @@ include::topics/making-open-source-more-inclusive.adoc[] | |
|
||
== Introduction | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Reorganized sections in guide:
|
||
// About the {UserCLIBookName} | ||
include::topics/about-cli-guide.adoc[leveloffset=+2] | ||
|
||
// About {ProductName} | ||
include::topics/mta-what-is-the-toolkit.adoc[leveloffset=+2] | ||
|
||
// About the {CLINameTitle} | ||
include::topics/about-cli.adoc[leveloffset=+2] | ||
|
||
// About the {UserCLIBookName} | ||
include::topics/about-cli-guide.adoc[leveloffset=+2] | ||
|
||
// Supported migration paths | ||
include::topics/migration-paths.adoc[leveloffset=+3] | ||
|
||
For more information about use cases and migration paths, see the link:https://developers.redhat.com/products/mta/use-cases[{ProductShortName} for developers] web page. | ||
|
||
// About the {CLINameTitle} | ||
include::topics/about-cli.adoc[leveloffset=+2] | ||
|
||
== Installing and Running the CLI | ||
|
||
// Install the CLI | ||
include::topics/installing-cli-tool.adoc[leveloffset=+2] | ||
|
||
// CLI known issues | ||
include::topics/cli-tool-known-issues.adoc[leveloffset=+3] | ||
include::topics/installing-cli-tool.adoc[leveloffset=+1] | ||
|
||
// Run the CLI | ||
include::topics/mta-cli-run.adoc[leveloffset=+2] | ||
include::topics/mta-cli-run.adoc[leveloffset=+1] | ||
|
||
// Analyze application source code | ||
include::topics/mta-cli-analyze.adoc[leveloffset=+3] | ||
|
@@ -52,11 +49,11 @@ include::topics/mta-cli-transform.adoc[leveloffset=+3] | |
// Available OpenRewrite recipes | ||
include::topics/available-openrewrite-recipes.adoc[leveloffset=+4] | ||
|
||
// Access the Report | ||
include::topics/access-report.adoc[leveloffset=+2] | ||
// Reports | ||
include::topics/access-report.adoc[leveloffset=+1] | ||
|
||
// Review the Reports | ||
include::topics/mta-review-reports.adoc[leveloffset=+1] | ||
include::topics/mta-review-reports.adoc[leveloffset=+2] | ||
|
||
// Export the Report in CSV Format | ||
// include::topics/csv-export.adoc[leveloffset=+1] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,11 +4,11 @@ | |
|
||
:_content-type: PROCEDURE | ||
[id="access-report_{context}"] | ||
= Accessing reports | ||
= Reports | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Changed section on opening an analysis report be chapter header. |
||
|
||
When you run the {ProductName}, a report is generated in the `<OUTPUT_REPORT_DIRECTORY>` that you specify using the `--output` argument in the command line. | ||
{ProductShortName} {CLIName} analyzes the application and generates a report of the analysis results. The report is saved to the `<OUTPUT_REPORT_DIRECTORY>` that you specified in the `./ analyze` command, after the `--output` argument. | ||
|
||
The output directory contains the following files and subdirectories: | ||
Navigate to the directory and open the `<APPLICATION_NAME>-output` directory, which contains the following files and subdirectories: | ||
|
||
---- | ||
<OUTPUT_REPORT_DIRECTORY>/ | ||
|
@@ -22,13 +22,13 @@ The output directory contains the following files and subdirectories: | |
|
||
.Procedure | ||
|
||
. Obtain the path of the `index.html` file of your report from the output that appears after you run {ProductShortName}: | ||
. In the `<APPLICATION_NAME>-output` directory that you specified in the {ProductShortName} `./analyze` command, open the `index.html` file: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is the Same question for line 9 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Corrected line 9 |
||
+ | ||
---- | ||
Report created: <OUTPUT_REPORT_DIRECTORY>/index.html | ||
Access it at this URL: file:///<OUTPUT_REPORT_DIRECTORY>/index.html | ||
---- | ||
|
||
. Open the `index.html` file by using a browser. | ||
. The `index.html` file opens in a browser by default. | ||
+ | ||
The generated report is displayed. | ||
The browser window shows the generated report. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,8 +7,7 @@ | |
[id="installing-cli-tool_{context}"] | ||
= Installing the {CLINameTitle} | ||
|
||
You can install the {CLINameTitle} on Linux, Windows, or macOS operating systems. | ||
|
||
You can install the {ProductShortName} {CLINameTitle} on Linux, Windows, or macOS operating systems. | ||
|
||
.Prerequisites | ||
|
||
|
@@ -24,24 +23,52 @@ Podman provides a command-line interface (CLI) familiar to anyone who has used t | |
For more information on installing and using Podman, see link:https://podman.io/docs/installation[Podman installation instructions]. | ||
==== | ||
|
||
.Installation options | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. New step on Installation options |
||
You can run either of the following two options to install {ProductShortName} {CLINameTitle}: | ||
|
||
* Downloading the appropriate {ProductShortName} {CLINameTitle} `.zip` file, according to your operating system, and installing by using the command line. | ||
* Running Podman on your computer and installing {ProductShortName} {CLIName} using Podman commands. | ||
|
||
+ | ||
[WARNING] | ||
==== | ||
Although installation using Podman is possible, downloading and installing the {ProductShortName} {CLIName} `.zip` file is the preferred installation. | ||
==== | ||
|
||
|
||
[id="installing-downloadable-cli-zip_{context}"] | ||
== Installing the {CLINameTitle} `.zip` file | ||
|
||
.Procedure | ||
|
||
To install using the downloadable `.zip` file: | ||
|
||
. Navigate to the link:{DevDownloadPageURL}[{ProductShortName} Download page] and download the OS-specific CLI file or the `src` file: | ||
. Navigate to the link:https://developers.redhat.com/products/mta/download[{ProductShortName} Download page] and download the OS-specific {ProductShortName} CLI file or the `mta-{ProductVersion}-cli-src` file: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added link to MTA CLI download page with all OS download options |
||
|
||
+ | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for Linux x86_64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for Linux aarch64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for macOS x86_64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for macOS aarch64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for Windows x86_64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for Windows aarch64 | ||
* Migration Toolkit {CLINameTitle} - {CLINameTitle} for {ProductShortName} source Code | ||
|
||
+ | ||
* {ProductShortNameLower}-{ProductVersion}-cli-linux.zip | ||
* {ProductShortNameLower}-{ProductVersion}-cli-macos.zip | ||
* {ProductShortNameLower}-{ProductVersion}-cli-windows.zip | ||
* {ProductShortNameLower}-{ProductVersion}-cli-src.zip | ||
[NOTE] | ||
==== | ||
Do not download and install the {ProductShortName} source code file if your computer runs one of the above operating systems. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. New note There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You need to use Vale 💡 RedHat.TermsSuggestions 1(101-106) Depending on the context, consider using 'earlier', 'previous', 'preceding', or 'before' rather than 'above'. |
||
==== | ||
|
||
. Extract the `.zip` file to a directory of your choice. The `.zip` file extracts a single binary, called *mta-cli*. | ||
+ | ||
When you encounter `<{ProductShortName}_HOME>` in this guide, replace it with the actual path to your {ProductShortName} installation. | ||
|
||
. Copy or save the `mta-cli` binary on Linux, `darwin-mta-cli on macOS, or `mta-cli.exe` on Windows, to a known directory. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nitpick: backtick missing after "darwin-mta-cli" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Corrected |
||
+ | ||
Continue to xref:cli-run_cli-guide[Running the CLI] to run the CLI command. | ||
|
||
|
||
[id="installing-using-podman_{context}"] | ||
== Installing the {CLINameTitle} using Podman | ||
|
@@ -60,10 +87,19 @@ To install using `podman pull`: | |
[source,terminal] | ||
---- | ||
podman login registry.redhat.io | ||
---- | ||
|
||
. Enter your `Username` and `Password`. | ||
+ | ||
[source,terminal] | ||
---- | ||
Username: <username> | ||
Password: <***********> | ||
---- | ||
. Issue: | ||
+ | ||
If your `Username` and `Password` are correct, the terminal line should show `Login Succeeded!`. | ||
|
||
. Copy the binary `PATH` for system-wide use. | ||
+ | ||
[source,terminal] | ||
---- | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,15 +4,15 @@ | |
|
||
:_content-type: PROCEDURE | ||
[id="cli-run_{context}"] | ||
= Running the {CLINameTitle} | ||
// = Running the {CLINameTitle} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Section upgraded to chapter heading |
||
|
||
You can run {ProductShortName} against your application. | ||
|
||
.Procedure | ||
|
||
. Open a terminal and navigate to the `<{ProductShortName}_HOME>/` directory. | ||
|
||
. Execute the `{mta-cli}` script, or `{mta-cli}.exe` for Windows, and specify the appropriate arguments: | ||
. Run the `{mta-cli}` script, or `{mta-cli}.exe` for Windows, or darwin-{mta-cli} for macOS, and specify the appropriate arguments: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nitpick: missing backticks around "darwin-{mta-cli}" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Corrected |
||
|
||
+ | ||
[source,terminal,subs="attributes+"] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,12 +17,12 @@ $ <{ProductShortName}_HOME>/bin/{mta-cli} --input /home/username/{mta-cli}-sourc | |
---- | ||
endif::cli-guide[] | ||
|
||
Use a browser to open the `index.html` file located in the report output directory. This opens a landing page that lists the applications that were processed. Each row contains a high-level overview of the story points, number of incidents, and technologies encountered in that application. | ||
Use a browser to open the `index.html` file located in the report output directory. This opens a landing page that lists the applications that were processed. Each row contains a high-level overview of the story points, number of incidents, and technologies encountered in that application. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The MTA analysis produces a static HTML file that shows the all the issues in the application. This is the same result as an analysis in the MTA Web Console, which should also cover the MTA report. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes please |
||
|
||
.Application list | ||
image::3-1-applications.png[Application list] | ||
|
||
NOTE: The incidents and estimated story points change as new rules are added to {ProductShortName}. The values here may not match what you see when you test this application. | ||
NOTE: The incidents and estimated story points change as new rules are added to {ProductShortName}. The values here might not match what you see when you test this application. | ||
|
||
The following table lists all of the reports and pages that can be accessed from this main {ProductShortName} landing page. Click the name of the application, *jee-example-app-1.0.0.ear*, to view the application report. | ||
|
||
|
@@ -78,7 +78,7 @@ The top navigation bar lists the various reports that contain additional details | |
| Provides a concise summary of all issues that require attention. | ||
|
||
| Application details | ||
| Provides a detailed overview of all resources found within the application that may need attention during the migration. | ||
| Provides a detailed overview of all resources found within the application that might need attention during the migration. | ||
|
||
| Technologies | ||
| Displays all embedded libraries grouped by functionality, allowing you to quickly view the technologies used in each application. | ||
|
@@ -141,7 +141,7 @@ This report includes details about every issue that was raised by the selected m | |
.Issues report | ||
image::3-4-issues-report.png[Issues report] | ||
|
||
Each reported issue may be expanded, by clicking on the title, to obtain additional details. The following information is provided. | ||
Each reported issue can be expanded, by clicking on the title, to obtain additional details. The following information is provided. | ||
|
||
* A list of files where the incidents occurred, along with the number of incidents within each file. If the file is a Java source file, then clicking the filename will direct you to the corresponding Source report. | ||
* A detailed description of the issue. This description outlines the problem, provides any known solutions, and references supporting documentation regarding either the issue or resolution. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why have you changed the .gitignore file?
You should be using your global .gitignore file which we set up in our last git training session.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have closed this comment and failed to correct your mistake