https://rancher.com/ logo
Join Slack
Powered by
This message was deleted.
# hobbyfarm
a
This message was deleted.
f
Yes! We also need to release a new chart version to fix some bugs (already merged but not released yet) Maybe this fixes your issues
c
Thanks. I created the PR, I can update the container images. My issue is: the ingress host name is changed automatically 😳 The definition is right but it's automatically changed... For example api.hobbyfarm.mydomain.com is changed to gargantua.hobbyfarm.mydomain.com. As a result the certificates are not generated
f
that seems weird. never heard of that issue... maybe @worried-fountain-60974 has an idea what is going on there?
and yes we would need the newsest master of ui/admin-ui/garg to be used for the newest image. and maybe we should release it as 2.1 (not yet 3.0)
c
Thanks for kicking the CI, I'm fixing the chart validation issue (not from me ^^)
f
Yes, thanks to you for doing this job
c
Here what happens (I reproduce it everytime). I'm looking at objects that could cause this.
Screenshare - 2023-03-07 3_22_33 PM.mp4
f
Okey i am not sure if this is related to hobbyfarm or to rancher...
c
Never have seen this before. I'll try on a local k3d
f
could you deploy it via the helm chart again and fill it in the values.yml? or have you tried this
c
Yes this is what I did I set the values
f
hmm. i have no clue to be honest
c
Thanks for looking at it, it helps me think about the possible causes
f
i would maybe remove the chart and install a fresh one again with the values.yml set to the correct value
c
I reinstalled with a new release name and url and a proper ingress class name and it worked \o/
f
Perfect!
c
Just admin/admin doesn't work on the UI Admin
I guess I need to create it manually, we could add it to the Helm chart, don't you think?
I made it optional @faint-optician-47536 if you have the time, the PR is good for me
f
Yes you need to add it manually due to security reasons
c
Sorry about this But I tried to be careful...
f
We removed that it is created by default
I will review it the next days
Unfortunately I only have some spare time on Friday, maybe only next week!
c
All the other Cloud Native components do add an admin, the install with Helm should be enough In this case it's disabled by default
Ok, no worries and sorry didn't want to take your time
f
Ok! Fine
c
As long as you can join this Tuesday it's fine by me 😄
f
Thank you for this work, we needed that !
Next Tuesday? Yes I think I can join
c
I have things to show...
Testing & stuff 😛
f
Okey I am looking forward
Oh btw. why have you removed namespace: {{ .Release.Namespace }} on the templates? Is this not needed anymore when installing helm charts?
c
Nop not needed 🙂
When we use Helm there is always a namespace defined for the installation (by default or can be specified)
f
okey i was not sure if helm always uses this. but then i am clear on that
c
I use as a reference the helm chart template we can create from the command line
f
okey i am letting the pipeline pass and will merge it afterwards
c
Thanks. I know I did many changes but it was the opportunity to make a clean version
f
ah wow github does not squash by default. i realize this everytime
c
For me I just need to select it once for a repo and then it remembers it
Don't know if it's user or repo based
f
maybe i forgot everytime i merge smth for the hobbyfarm/hobbyfarm 😄
it is repo based. i have done this multiple times in other repos
it released the new chart version 2.0.7 automatically as it seems
ah okey you changed it to 2.0.7. We mayneed to update the images as well
c
Even if the images didn't change some yaml files did, at least the ingressClassName
I looked for new images but none with a specific version
f
yes sure.
yes we have not released new versions of admin-ui/ui/garg for now
we still have so many things to implement for a nice version 3 😛
🙂 1
c
Don't know when we decide to deliver a specific version. We create latest but I think it's tag based
f
yes but latest may contain some bugs
we will release a new version in the future. but for now users are fine with v2
Open in Slack
PreviousNext
Powered by