coderabbitai[bot]
commented
1 week ago [!WARNING]
Rate limit exceeded
@bupd has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 46 minutes and 9 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between fa67a73441206c727f750c6a51ca4211e8919776 and 2cf1671df233f6f80f5b9c6f7cbeecf14dc5297d.
Walkthrough
The updates introduce a new feature called "Harbor Satellite" aimed at decentralizing Harbor container registries for edge computing environments. Key changes include configuration and setup files, code implementations for building and running the satellite service, handling image replication and storage, and end-to-end tests. These modifications enable Harbor systems to function reliably in scenarios with intermittent internet connectivity by storing and serving container images locally.
Changes
| Files | Change Summary |
|---|---|
.env, .gitignore, config.toml, config.json |
Added configuration files for environment variables, registry settings, and gitignore updates to exclude .env file. |
README.md |
Updated to reflect the new "Harbor Satellite" project details, use cases, background, and proposal. |
ci/main.go, main.go |
Introduced constants, new functions for satellite service, and configuration setup using viper. |
go.mod |
Updated and added module dependencies for handling container registries and environment variables. |
image-list/images.json |
New JSON file for storing image repository information. |
internal/replicate/replicate.go |
Refactored functions to handle different image schemes and improved image reference handling. |
internal/satellite/satellite.go |
Added functions for creating and running Satellite instances with error handling and immediate execution logic. |
internal/store/… |
Introduced in-memory store, file fetch, and HTTP fetch functionality for managing images. |
registry/launch-registry.go |
New function for initializing and launching the registry server. |
test/e2e/… |
New end-to-end test files for setting up registries, pushing images, and building the satellite service using Dagger. |
Sequence Diagram(s)
sequenceDiagram
participant User
participant Satellite
participant Registry
User->>Satellite: Start Satellite Service
activate Satellite
Satellite->>Registry: Fetch Initial Configurations
Registry-->>Satellite: Configurations
User->>Satellite: Request Image
Satellite->>Registry: Retrieve Image
Registry-->>Satellite: Image Data
Satellite-->>User: Serve Image
deactivate Satellite
Note over Satellite: Operates independently if registry is unavailablePoem
In the wired world where data's streamed,
Harbor Satellite ensures dreams, At the edge where signals wane,
A beacon stands amidst the rain.
Images stored with care so neat,
Ensuring service, never beat.
Cheers to code, now furrows light,
Harbor's edge, forever bright! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
bupd
Overview:
This PR adds end-to-end testing with dagger.
Changes Introduced:
Prerequisites:
Tests require docker to be installed in the system.
How to Test:
go test -v ./test/e2e/registry_test.go.Fixes: #26
Summary by CodeRabbit
New Features
Documentation
Chores
.envand.gitignorefor environment configuration and version control.config.tomlandconfig.json) for registry and testing settings.Tests