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.

Sign up for GitHub

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

Jump to bottom

fix(nexus): add use_artifact as passthrough messages #6447

Merged
merged 8 commits into from
Oct 12, 2023
Merged

fix(nexus): add use_artifact as passthrough messages #6447

merged 8 commits into from
Oct 12, 2023

Conversation

kptkin
Copy link
Contributor

@kptkin kptkin commented Oct 12, 2023 •
edited

Description

What does the PR do?

🤖 Generated by Copilot at 2a54703

This pull request adds support for tracking and uploading artifact usage and summary metrics for runs. It modifies sender.go to store and send a JSON file with the summary data, and handler.go to handle the UseArtifact record type.

Testing

How was this PR tested? Enabled skipped tests

🤖 Generated by Copilot at 2a54703

Sing, O Muse, of the code that the heroes wrought
To store and send the deeds of their runs sublime,
How they crafted a file with the name SummaryFilename
To hold the metrics of their toils and their time.

@kptkin kptkin changed the title Use artifact nexus fix(nexus): add use_artifact as passthrough messages Oct 12, 2023
@github-actions github-actions bot added cc-fix and removed cc-fix labels Oct 12, 2023
@dmitryduev dmitryduev marked this pull request as ready for review October 12, 2023 19:49
@dmitryduev dmitryduev requested a review from a team as a code owner October 12, 2023 19:49
dmitryduev
dmitryduev approved these changes Oct 12, 2023
View reviewed changes
Copy link
Member

@dmitryduev dmitryduev left a comment

Choose a reason for hiding this comment

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

lgtm!

@github-actions github-actions bot added cc-fix and removed cc-fix labels Oct 12, 2023
@kptkin kptkin merged commit d08e6ab into main Oct 12, 2023
67 of 69 checks passed
@kptkin kptkin deleted the use-artifact-nexus branch October 12, 2023 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmitryduev dmitryduev dmitryduev approved these changes

Assignees
No one assigned
Labels
cc-fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kptkin @dmitryduev