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

[Task] Add "test connection" button to Custom Target creation modal #594

Closed
Tracked by #592
andrewazores opened this issue Oct 28, 2022 · 3 comments · Fixed by #891
Closed
Tracked by #592

[Task] Add "test connection" button to Custom Target creation modal #594

andrewazores opened this issue Oct 28, 2022 · 3 comments · Fixed by #891
Assignees
Labels
feat New feature or request

Comments

@andrewazores
Copy link
Member

andrewazores commented Oct 28, 2022

As a user, when I'm defining a new custom target it would be useful to be able to verify that the connection URL I've entered is correct by having a "test connection" button that I can click before actually pressing "Submit". Right now, the only way to do this is by creating the custom target definition and then going to a view like Recordings to see if I'm able to retrieve and view the Active Recordings. If I entered the URL wrong I then have to delete the custom target, create a new one, and try again. Having a "test connection" button would drastically shorten that feedback loop and help catch input errors sooner.

@tthvo
Copy link
Member

tthvo commented Feb 8, 2023

@andrewazores I am wondering why endpoint for creating custom target is not accepting labels similar to annotations? All custom targets now have empty labels...

@andrewazores
Copy link
Member Author

There are basically three kind of key-value stores attached to targets:

  • labels
  • annotations.platform
  • annotations.cryostat

Labels are implicitly meant to come from the deployment platform, along with the platform annotations. ie. in a k8s environment with targets defined by the KubeApiPlatformClient, those two fields are copied from the k8s Endpoints/Pods objects etc.

Cryostat annotations are the ones that add extra information from the application, or in this case the client/user.

Now that there is the Discovery Plugin API it's possible for a client to register itself in a way that allows it to publish its own labels and platform annotations however it pleases too.

I think it makes sense for the labels and platform annotations to continue to be "immutable" (or at least defined by a plugin that probably keeps them stable), whereas the Cryostat annotations can change arbitrarily including by interactive user intervention.

@tthvo
Copy link
Member

tthvo commented Feb 8, 2023

Ahh right that makes sense! thanks for explaining :D

@tthvo tthvo mentioned this issue Feb 21, 2023
42 tasks
@tthvo tthvo linked a pull request Feb 21, 2023 that will close this issue
42 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Done in 2.3.0 release Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants