odpi / egeria-charts

Helm chart repository
https://odpi.github.io/egeria-charts
Apache License 2.0
13 stars 9 forks source link

3.14.0 prerelease 0 #225

Closed lpalashevski closed 1 year ago

lpalashevski commented 1 year ago

Signed-off-by: Ljupcho Palashevski lpalashevski@gmail.com

planetf1 commented 1 year ago

Re the charts - added some notes to https://github.com/odpi/egeria-jupyter-notebooks/pull/35 - wonder if more notebooks need the update

lpalashevski commented 1 year ago

Re the charts - added some notes to odpi/egeria-jupyter-notebooks#35 - wonder if more notebooks need the update

I will test what is in scope for the release (building-data-catalog and open-lineage labs) if I spot other things breaking I will fix/raise issue.

But first I need this pre-release available in the helm repo (I do not want to have this tested form local chart)

planetf1 commented 1 year ago

I will test what is in scope for the release (building-data-catalog and open-lineage labs) if I spot other things breaking I will fix/raise issue.

More are affected. I opened up https://github.com/odpi/egeria-jupyter-notebooks/issues/36 and can make the fixes - can decide if we want them in release or ready for next. Good spot.

https://github.com/odpi/egeria-jupyter-notebooks/issues/36

But first I need this pre-release available in the helm repo (I do not want to have this tested form local chart) Agreed. The only outstanding conversation is the change to graph repo. It's not critical for a prerelease, but wondered if it was intentional or accidental.

For testing in non-default config, using '--set value=xxx' or creating a .yml file with the overrides is a good approach I'd recommend.