Skip to content
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-3588: Release Notes MTA 7.2.0 #67

Open
wants to merge 34 commits into
base: main
Choose a base branch
from

Conversation

anarnold97
Copy link
Collaborator

@anarnold97 anarnold97 commented Oct 17, 2024


In {ProductFullName} 7.1.0, *Preemption* cannot be enabled for tasks in the Task Manager page and the Task Manager drawer if {ProductShortName} authentication is enabled. To work around this issue, disable authentication. link:https://issues.redhat.com/browse/MTA-3195[(MTA-3195)]

.CLI analysis works extremely slow on Windows
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibragins - MTA-3667 added to the release notes. Is there anything else you would like me to add?
Thanks

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you, that is it

Copy link
Collaborator

@istein1 istein1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good.

You can perform an application analysis for Java applications by using the {ProductShortName} CLI that does not require installation of a container runtime.

// add link once docs published
// :FeatureName: Support for analyzing applications managed with Gradle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was new in MTA-7.1


.Containerless CLI

You can perform an application analysis for Java applications by using the {ProductShortName} CLI that does not require installation of a container runtime.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A suggestion: Direct to the CLI guide to see more details on this feature.

Copy link
Collaborator Author

@anarnold97 anarnold97 Dec 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do when we have published. As we have not yet merged the Containerless CLI docs, the link would break the build in Pantheon. I need to publish Masha's doc and then generate the link

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When linking the CLI, please specify that the user has some installation prereqs to satisfy. I.e..

"See to learn more about installation requirements" or something like that

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do this after publishing.

docs/topics/ref_resolved-issues-7-2-0.adoc Outdated Show resolved Hide resolved

This section provides upgrade notes for the {ProductFullName} 7.2.0.

// Information waiting for Nandini
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nachandr ,
Is there some content you need to add here please?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nandini did reply - Nothing needs to be included.

see - Slack discussion

Section commented out in master.adoc

@anarnold97 anarnold97 force-pushed the MTA-release-notes-7.2.0 branch from 5ff70f8 to 0a83db7 Compare December 8, 2024 12:38

.CLI analysis is extremely slow on Windows

{ProductShortName} CLI analysis does not function as expected on Microsoft Windows, with analysis using container runtimes being extremely slow on Windows. This issue is planned to be resolved in {ProductShortName} 7.2.1. link:https://issues.redhat.com/browse/MTA-3667[(MTA-3667)]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibragins - FYI


.CLI bulk analysis does not function on Windows

{ProductShortName} CLI bulk analysis, performed by using the `--bulk` option, does not function on Microsoft Windows. This issue is planned to be resolved in {ProductShortName} 7.2.1. link:https://issues.redhat.com/browse/MTA-4307[(MTA-4307)]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibragins - FYI


In previous versions of {ProductShortName}, the bulk analysis on Windows systems could not be performed. This issue has been resolved now in the {ProductShortName} CLI. link:https://issues.redhat.com/browse/MTA-3624[(MTA-3624)]

.IntelliJ IDEA cloud target is not functioning as expected
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rename this to "IntillJ IDE target list showed unusable targets"

This was actually impacting other issues too would be good to call out

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dymurray - please can you send me a list of other issues and i will add as bullet points underneath:

This impacts other issues, including:
* 

Copy link
Contributor

@dymurray dymurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment in slack for @anarnold97 but aside from that, LGTM!


{ProductShortName} CLI analysis fails with a generic error when incorrect Maven credentials are used. The failure occurs because the analyzer is unable to report detailed error messages. link:https://issues.redhat.com/browse/MTA-3714[(MTA-3714)]

.{ProductShortName} CLI returns confusing error messages when Python is not installed on Windows
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibragins - FYI


{ProductShortName} CLI returns two confusing error messages of `failed to validate flags` and `exit status 9009` when Python is not installed on Microsoft Windows. This issue is planned to be resolved in {ProductShortName} 7.2.1. link:https://issues.redhat.com/browse/MTA-4319[(MTA-4319)]

.{ProductShortName} Containerless CLI Java remains in memory after analysis is completed
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibragins - FYI

@anarnold97 anarnold97 requested a review from ibragins December 9, 2024 21:12
Copy link
Collaborator

@ibragins ibragins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @anarnold97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants