-
Notifications
You must be signed in to change notification settings - Fork 36
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 imagefactory docker compose example #58
base: main
Are you sure you want to change the base?
Conversation
SCHEMATICS_SLEEP_TIME=600 | ||
|
||
IMGFAC_VERSION=v0.5.0 | ||
IMGFAC_EXTERNAL_URL=imgfac.example.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be a proper URL (when passed as an argument), this might be the reason the UI doesn't work properly, e.g. http://imgfac.example.com/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd argue since no reverse-proxy is part of this scenario, we can assume it is always http
.
See 9d937af
crane push \ | ||
--insecure \ | ||
${TMP_FILE} \ | ||
${REGISTRY_URL}/installer/${SCHEMA_ID}:${TALOS_VERSION} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why this is needed, Image Factory itself can work as a container registry (to pull installer images from)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't really remember to be honest. I wrote this code some while ago and I wasn't aware the registry I added here is actually not needed, I thought imagefactory needs this as a companion, it made sense in my head I guess.
Do you want me to remove the external registry (called registry
in docker-compose.yaml
) and related code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This registry should be there, but there's no need to push anything there. All I'm asking is to remove this file completely, as it's not needed
9d937af
to
33556f4
Compare
|
||
## URLs not working | ||
|
||
There is a tiny problem in the `imagefactory` frontend: The URLs generated contain the external domain used and it is duplicated for some reason. This is particularly mean because the URL _visible_ in the UI looks correct, but the `HTML` `href` is not. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this should be fixed is the proper URL (with http://
is passed)
This code deploys
sidero
imagefactory in a localdocker compose
setup.It leaves out TLS, but deploys an upstream
ghcr.io
mirror and anotherregistry
that generated images are pushed to. It also ships a tiny script that generates images from predefined schemas.see
README.md
for more details.related slack message