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

[deckhouse] limit pulled modules #11909

Merged
merged 4 commits into from
Feb 6, 2025
Merged

[deckhouse] limit pulled modules #11909

merged 4 commits into from
Feb 6, 2025

Conversation

ipaqsa
Copy link
Member

@ipaqsa ipaqsa commented Feb 4, 2025 •
edited
Loading

Description

It limits tags.

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

To avoid server overwhelming.

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: deckhouse
type: fix
summary: Limit pulled modules.
impact_level: low

@ipaqsa
[deckhouse] fix list tags page size
dac7105 
Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>
@ipaqsa ipaqsa self-assigned this Feb 4, 2025
@github-actions github-actions bot added area/core Pull requests that update core modules go Pull requests that update Go code labels Feb 4, 2025
@ipaqsa ipaqsa added this to the v1.68.0 milestone Feb 4, 2025
@ipaqsa ipaqsa requested a review from yalosev February 4, 2025 21:53
@ipaqsa
[deckhouse] limit pulled modules len
7e149a9 
Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>
@ipaqsa ipaqsa changed the title [deckhouse] fix list tags page size [deckhouse] limit pulled module Feb 4, 2025
@ipaqsa ipaqsa marked this pull request as ready for review February 4, 2025 22:06
@ipaqsa ipaqsa changed the title [deckhouse] limit pulled module [deckhouse] limit pulled modules Feb 4, 2025
@ipaqsa
[deckhouse] up default timeout
3cd64d9 
Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>
@ipaqsa
[deckhouse] up limit
1f8eab5 
Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>
@yalosev yalosev merged commit d9e8597 into main Feb 6, 2025
36 of 37 checks passed
@yalosev yalosev deleted the fix/tags-page-size branch February 6, 2025 21:53
Paramoshka pushed a commit that referenced this pull request Feb 18, 2025
@ipaqsa @Paramoshka
[deckhouse] limit pulled modules (#11909)
ea4d6ff 
Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yalosev yalosev yalosev approved these changes

@miklezzzz miklezzzz miklezzzz approved these changes

@ldmonster ldmonster Awaiting requested review from ldmonster ldmonster is a code owner

Assignees

@ipaqsa ipaqsa

Labels
area/core Pull requests that update core modules go Pull requests that update Go code
Projects
None yet
Milestone
v1.68.0
Development

Successfully merging this pull request may close these issues.
3 participants
@ipaqsa @yalosev @miklezzzz