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.

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

feat: Add images and demo script for in-cluster registry #801

Merged
merged 4 commits into from
Nov 6, 2024

Conversation

jimmidyson
Copy link
Contributor

Would love it if someone would try it out locally and tell me it works for them too!

This will be used when utilising mindthegap to bootstrap an in-cluster
registry.
This will be used when utilising mindthegap to bootstrap an in-cluster
registry.
Copy link
Contributor

Unit test results

100 tests  ±0   100 ✅ ±0   0s ⏱️ ±0s
 25 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 759de52. ± Comparison against base commit ba97fb2.

Copy link
Contributor

e2e test results

34 tests  ±0   31 ✅ ±0   1m 28s ⏱️ +2s
 2 suites ±0    3 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 759de52. ± Comparison against base commit ba97fb2.

Copy link
Contributor

@dkoshkin dkoshkin left a comment

Choose a reason for hiding this comment

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

Nice! ( worked locally for me after I figured out we're no longer using make :| )

For any further help, visit https://docs.cilium.io/en/v1.16/gettinghelp
node/self-hosted-registry-demo-control-plane condition met
node/self-hosted-registry-demo-worker condition met
NAME                                      STATUS   ROLES           AGE   VERSION
self-hosted-registry-demo-control-plane   Ready    control-plane   58s   v1.27.3
self-hosted-registry-demo-worker          Ready    <none>          35s   v1.27.3
pod "busybox" deleted
error: timed out waiting for the condition
Yay everything worked as expected - this was supposed to fail!

cmd/copy-file-to-pod/main.go Show resolved Hide resolved
hack/self-hosted-registry-demo/demo.sh Show resolved Hide resolved
Copy link
Contributor

@dkoshkin dkoshkin left a comment

Choose a reason for hiding this comment

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

Great demo script!

@jimmidyson jimmidyson merged commit f6f0747 into main Nov 6, 2024
14 checks passed
@jimmidyson jimmidyson deleted the jimmi/mindthegap-cluster-bootstrap branch November 6, 2024 17:16
jimmidyson pushed a commit that referenced this pull request Nov 11, 2024
🤖 I have created a release *beep* *boop*
---


## 1.17.0 (2024-11-11)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

## What's Changed
### Exciting New Features 🎉
* feat: Add images and demo script for in-cluster registry by
@jimmidyson in #801
* feat: Build with go 1.23.3 and upgrade all tools by @jimmidyson in
#808
### Fixes 🔧
* fix: Allow TLS to pull chart from OCI repository by @dlipovetsky in
#804
* fix: Reload registry TLS certificate when it is updated on the file
system by @dlipovetsky in
#805

## New Contributors
* @dlipovetsky made their first contribution in
#804

**Full Changelog**:
v1.16.0...v1.17.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: mesosphere-actions-pr-bot[bot] <157582460+mesosphere-actions-pr-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants