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

feat(group): increase logging for grouping #609

Merged
merged 2 commits into from
Aug 17, 2023

Conversation

blacha
Copy link
Member

@blacha blacha commented Aug 17, 2023

Why?

When artifact errors occur in argo its hard to figure out exactly what files were written and what is in those files.

Description

This increases the logging to explain exactly which files were written and some information about their contents.

When running argo containers having the exact github action that made the container is very useful, we were loosing the -attempt number so it was not possible to get the exact version number.

@blacha blacha added this pull request to the merge queue Aug 17, 2023
Merged via the queue into master with commit 2966586 Aug 17, 2023
1 check passed
@blacha blacha deleted the feat/increase-group-logging branch August 17, 2023 03:31
github-merge-queue bot pushed a commit that referenced this pull request Sep 6, 2023
🤖 I have created a release *beep* *boop*
---


##
[2.13.0](v2.12.0...v2.13.0)
(2023-09-05)


### Features

* **cli:** Move the basemaps github create-pr cli into argo-tasks.
BM-872 ([#627](#627))
([a47b38a](a47b38a))
* **group:** increase logging for grouping
([#609](#609))
([2966586](2966586))


### Bug Fixes

* **copy:** copy and fix content-types TDE-859
([#633](#633))
([cc7c5cc](cc7c5cc))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants