akuity / kargo

Application lifecycle orchestration
https://kargo.akuity.io/
Apache License 2.0
1.55k stars 133 forks source link

feat: limit page requested page size to 100 for list tags requests to quay.io #2186

Closed krancour closed 3 months ago

krancour commented 3 months ago

Quay is recently returning a hard error if a page size greater than 100 is requested.

netlify[bot] commented 3 months ago

Deploy Preview for docs-kargo-akuity-io ready!

Name Link
Latest commit 8cdea72c269bbfc2ee8902651ea0678a930120eb
Latest deploy log https://app.netlify.com/sites/docs-kargo-akuity-io/deploys/6675d53082acd400084c0803
Deploy Preview https://deploy-preview-2186.kargo.akuity.io
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 46.30%. Comparing base (cc9f958) to head (8cdea72). Report is 1 commits behind head on main.

Files Patch % Lines
internal/image/repository_client.go 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2186 +/- ## ========================================== - Coverage 46.31% 46.30% -0.01% ========================================== Files 242 242 Lines 16763 16770 +7 ========================================== + Hits 7763 7766 +3 - Misses 8629 8633 +4 Partials 371 371 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

krancour commented 3 months ago

Quay has backed out the change that prompted this PR. Closing.