Skip to content

Conversation

mat1e
Copy link
Member

@mat1e mat1e commented Aug 19, 2022

Link to #150

This PR add somes metrics to existing ones and add improvments to existing functionnality.

Commons Points :

  • instance : Jenkins instance url
  • project_namespace : root path to the project (parent folder)

Jenkins Points :

  • build_user : user who launch the build
  • build_cause : Trigger type

Fix and improvments :

  • Research of node_name in logs if the env var isn't setted
  • Fix NoClassDefFoundException in tests by adding sshd plugin (it was a module before, see JEP-230)

@asimell
Copy link
Contributor

asimell commented Aug 29, 2022

Please also add relevant documentation to the available metrics documentation.

Copy link
Contributor

@asimell asimell left a comment

Choose a reason for hiding this comment

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

LGTM, but there's some merge conflicts. Could you address those before I merge this?

@asimell asimell merged commit 073b4c1 into jenkinsci:development Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants