Skip to content

fix(api): block device WaitForUserUpload phase#2178

Open
hardcoretime wants to merge 1 commit intomainfrom
fix/vd/uploader
Open

fix(api): block device WaitForUserUpload phase#2178
hardcoretime wants to merge 1 commit intomainfrom
fix/vd/uploader

Conversation

@hardcoretime
Copy link
Copy Markdown
Contributor

@hardcoretime hardcoretime commented Apr 1, 2026

Description

  • Added a GET method to the upload URL to signal that the upload server is ready to serve.
  • Added a readiness probe to the uploader pod.
  • Fixed missing output from the data export in the E2E test case.
  • Fixed the WaitForUserUpload phase for block devices.

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

The WaitForUserUpload phase for block devices was being set incorrectly because the ingress controller requires some time to apply the configuration from the uploader ingress resource. This delay can cause 503 and 413 status codes on upload operations if they start before the ingress configuration is applied.

What is the expected result?

  • The uploader pod is ready when the readiness probe is successful.
  • The WaitForUserUpload phase is set when the uploader server returns an HTTP status OK.

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: api
type: fix
summary: "Enhanced reliability of the upload server for block devices by adding readiness checks and fixing configuration delays to prevent upload errors."

@hardcoretime hardcoretime added this to the v1.7.0 milestone Apr 1, 2026
@hardcoretime hardcoretime force-pushed the fix/vd/uploader branch 3 times, most recently from 0706d9b to bb9c27a Compare April 2, 2026 07:37
@nevermarine nevermarine modified the milestones: v1.7.0, v1.8.0 Apr 3, 2026
@hardcoretime hardcoretime force-pushed the fix/vd/uploader branch 2 times, most recently from 33c6711 to dbdb022 Compare April 3, 2026 17:09
Signed-off-by: Roman Sysoev <roman.sysoev@flant.com>
@hardcoretime hardcoretime changed the title fix(vd): uploader fix(api): block device WaitForUserUpload phase Apr 3, 2026
@hardcoretime hardcoretime added the e2e/run Run e2e test on cluster of PR author label Apr 3, 2026
@deckhouse-BOaTswain
Copy link
Copy Markdown
Contributor

deckhouse-BOaTswain commented Apr 3, 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 Apr 3, 2026
@hardcoretime hardcoretime marked this pull request as ready for review April 4, 2026 07:41
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