Skip to content

fix(dvcr): improve importer logging#2016

Draft
hardcoretime wants to merge 3 commits intomainfrom
fix/dvcr/improve-importer-logging
Draft

fix(dvcr): improve importer logging#2016
hardcoretime wants to merge 3 commits intomainfrom
fix/dvcr/improve-importer-logging

Conversation

@hardcoretime
Copy link
Contributor

@hardcoretime hardcoretime commented Feb 20, 2026

Description

  • Ensure upload layer errors are not missed.
  • Wrap "DVCR is out of space" error.

Why do we need it, and what problem does it solve?

This is required to improve user experience because the current message is not clear enough to indicate that DVCR is out of space.

What is the expected result?

When DVCR is out of space, a resource will contain a clearer message about it in the condition.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: dvcr
type: fix
summary: "When DVCR is out of space, a resource will contain a clearer message about it in the condition."

@hardcoretime hardcoretime added this to the v1.6.0 milestone Feb 20, 2026
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch 5 times, most recently from 47db2cc to 7a7c810 Compare February 24, 2026 13:19
@hardcoretime hardcoretime marked this pull request as ready for review February 24, 2026 13:40
@hardcoretime hardcoretime added e2e/run Run e2e test on cluster of PR author labels Feb 24, 2026
@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 24, 2026
@hardcoretime hardcoretime added the e2e/run Run e2e test on cluster of PR author label Feb 24, 2026
@deckhouse-BOaTswain
Copy link
Contributor

Workflow has started.
Follow the progress here: Workflow Run

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 24, 2026
@hardcoretime hardcoretime added the e2e/run Run e2e test on cluster of PR author label Feb 24, 2026
@deckhouse-BOaTswain
Copy link
Contributor

Workflow has started.
Follow the progress here: Workflow Run

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 24, 2026
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch 2 times, most recently from 71532b1 to 8078ece Compare February 25, 2026 14:06
@hardcoretime hardcoretime marked this pull request as draft February 26, 2026 07:17
Roman Sysoev added 2 commits February 26, 2026 17:50
- Ensure upload layer errors are not missed.
- Wrap "DVCR is out of space" error.

Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
- fix ide import auto formatting
- wrap chan errors
- improve naming
- use context cancel func

Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch from 9bc4c2f to 8390dc3 Compare February 26, 2026 14:50
@hardcoretime hardcoretime added the build/github/ubuntu Run jobs on github runners label Feb 26, 2026
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch 2 times, most recently from 5b6da4c to e98cf09 Compare February 26, 2026 18:00
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch from e98cf09 to 74a6502 Compare February 26, 2026 22:22
@hardcoretime hardcoretime added e2e/run Run e2e test on cluster of PR author and removed build/github/ubuntu Run jobs on github runners labels Feb 26, 2026
@deckhouse-BOaTswain
Copy link
Contributor

deckhouse-BOaTswain commented Feb 26, 2026

Workflow has started.
Follow the progress here: Workflow Run

The target step completed with status: failure.

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 26, 2026
@hardcoretime hardcoretime added the e2e/run Run e2e test on cluster of PR author label Feb 26, 2026
@deckhouse-BOaTswain
Copy link
Contributor

deckhouse-BOaTswain commented Feb 26, 2026

Workflow has started.
Follow the progress here: Workflow Run

The target step completed with status: failure.

@deckhouse-BOaTswain deckhouse-BOaTswain removed the e2e/run Run e2e test on cluster of PR author label Feb 27, 2026
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime force-pushed the fix/dvcr/improve-importer-logging branch from 66b6bd6 to 591d159 Compare February 27, 2026 09:50
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