Skip to content

Remove unused SYNC_QUOTA from env.jinja template#22280

Merged
wy65701436 merged 2 commits intogoharbor:mainfrom
mxab:patch-1
Sep 1, 2025
Merged

Remove unused SYNC_QUOTA from env.jinja template#22280
wy65701436 merged 2 commits intogoharbor:mainfrom
mxab:patch-1

Conversation

@mxab
Copy link
Copy Markdown
Contributor

@mxab mxab commented Aug 25, 2025

SYNC_QUOTA is not used throughout the whole code.
It looks like the feature using it was removed.

This PR removes the SYNC_QUOTA line from make/photon/prepare/templates/core/env.jinja

Issue being fixed

Fixes #22279

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

`SYNC_QUOTA` is not used throughout the whole code.

It looks like the feature using it was removed.

Signed-off-by: Max Fröhlich <maxbruchmann@gmail.com>
@mxab mxab requested a review from a team as a code owner August 25, 2025 10:34
Copy link
Copy Markdown
Contributor

@bupd bupd left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.88%. Comparing base (c8c11b4) to head (e6cfdf7).
⚠️ Report is 550 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #22280       +/-   ##
===========================================
+ Coverage   45.36%   65.88%   +20.51%     
===========================================
  Files         244     1072      +828     
  Lines       13333   115932   +102599     
  Branches     2719     2927      +208     
===========================================
+ Hits         6049    76380    +70331     
- Misses       6983    35319    +28336     
- Partials      301     4233     +3932     
Flag Coverage Δ
unittests 65.88% <ø> (+20.51%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 986 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@wy65701436 wy65701436 added the release-note/update Update or Fix label Sep 1, 2025
@wy65701436 wy65701436 merged commit 26b6139 into goharbor:main Sep 1, 2025
12 checks passed
AYDEV-FR pushed a commit to AYDEV-FR/harbor that referenced this pull request Sep 19, 2025
Remove SYNC_QUOTA from env.jinja template

`SYNC_QUOTA` is not used throughout the whole code.

It looks like the feature using it was removed.

Signed-off-by: Max Fröhlich <maxbruchmann@gmail.com>
Co-authored-by: Wang Yan <wangyan@vmware.com>
AYDEV-FR pushed a commit to AYDEV-FR/harbor that referenced this pull request Sep 19, 2025
Remove SYNC_QUOTA from env.jinja template

`SYNC_QUOTA` is not used throughout the whole code.

It looks like the feature using it was removed.

Signed-off-by: Max Fröhlich <maxbruchmann@gmail.com>
Co-authored-by: Wang Yan <wangyan@vmware.com>
AYDEV-FR pushed a commit to AYDEV-FR/harbor that referenced this pull request Sep 19, 2025
Remove SYNC_QUOTA from env.jinja template

`SYNC_QUOTA` is not used throughout the whole code.

It looks like the feature using it was removed.

Signed-off-by: Max Fröhlich <maxbruchmann@gmail.com>
Co-authored-by: Wang Yan <wangyan@vmware.com>
OrlinVasilev pushed a commit to OrlinVasilev/harbor that referenced this pull request Oct 29, 2025
Remove SYNC_QUOTA from env.jinja template

`SYNC_QUOTA` is not used throughout the whole code.

It looks like the feature using it was removed.

Signed-off-by: Max Fröhlich <maxbruchmann@gmail.com>
Co-authored-by: Wang Yan <wangyan@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove SYNC_QUOTA from env template

7 participants