Skip to content

Adding Diagnosis command to cli - #415

Closed
dashbhanu wants to merge 1 commit into
jfrog:masterfrom
dashbhanu:diagnostics
Closed

Adding Diagnosis command to cli#415
dashbhanu wants to merge 1 commit into
jfrog:masterfrom
dashbhanu:diagnostics

Conversation

@dashbhanu

@dashbhanu dashbhanu commented Jul 3, 2019

Copy link
Copy Markdown

A new features in CLI to get a diagnosis of Artifactory after each upgrade. This command should run before and after upgrade, so that it will print the performance and stability of artifactory instance. The default command is like
jfrog rt diagnosis

@eyalbe4

eyalbe4 commented Jul 16, 2019

Copy link
Copy Markdown
Contributor

@dashbhanu,
First, thanks for working on this!

I have to say that I'm having issues with the basic design of this functionality. There's a lot I think we need to modify, so I'm starting from some of the most significant points:

Reading plain text (not formatted) response and then parsing it manually, is something we really want to avoid.
It is okay receive a JSON responses and then reading it into a struct, but parsing strings is not the way to go from our perspective. In addition, the content is currently saved to files as a stream of bytes, which make it unreadable.

I don't think that the code should be included as part of the jfrog-client-go library, because I find it hard it believe that it is useful for other applications - at least they way the API is built now.

There's a lot more to discuss, in terms of where the info files should be saved to, how the command should receive them, code styling and more, but I suggest we start with the above first.

Thanks

@dashbhanu

Copy link
Copy Markdown
Author

Thank you @eyalbe4 for your time, I will take some more time and go throw your review comments and then again resubmit PR.

Thanks

@eyalbe4

eyalbe4 commented Nov 24, 2019

Copy link
Copy Markdown
Contributor

@dashbhanu,
Would you like to continue working on this?

@eyalbe4

eyalbe4 commented Jan 5, 2020

Copy link
Copy Markdown
Contributor

Closing this one for now.

@eyalbe4 eyalbe4 closed this Jan 5, 2020
bhanurp added a commit that referenced this pull request Sep 2, 2026
build-info-go's NuGet FlexPack work landed on main as b325d34 (squash of
PR #415), so the branch pseudo-version is no longer needed. Also bumps
jfrog-cli-artifactory to caf4135, which makes the same change.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

3 participants