eclipse-tractusx / digital-product-pass

digital product pass
https://github.com/eclipse-tractusx/digital-product-pass
Apache License 2.0
22 stars 13 forks source link

QG 4 checks Release 24.03 #180

Closed saudkhan116 closed 6 months ago

saudkhan116 commented 7 months ago

QG checks

Please keep this issue open until QG 24.03 is concluded and will be managed by the Issue Creator! We will inform you about finding and proposals in separated issues, this issue here is for the Overview of the Checks!

Please keep this issue open until QG is concluded!

Product Name: Digital Product Pass Dev SPOC: @matbmoser , @saudkhan116, @davidzynda, @dsrparracho Helm Chart Version: v2.1.4 App Version: v2.1.3 -> https://github.com/eclipse-tractusx/digital-product-pass/releases/tag/v2.1.3 QG5 Approval: yes/no

Release Managemnet Reference Issue: https://github.com/eclipse-tractusx/sig-release/issues/511

Check of Tractus-X Release Guidelines

TRG 1 Documentation

TRG 2 Git

TRG 3 Kubernetes

TRG 4 Container

TRG 5 Helm

Checks within TRG 5.01 - [x] TRG 5.01 appropriate semantic versioning for `version` and `appVersion` has to be used in `Chart.yaml` - [x] #186 - [x] TRG 5.01 `values.yaml` file must contain proper default values/placeholders - [x] TRG 5.01 No hostname provided for ingress - [x] TRG 5.01 Ingress is disabled - [x] TRG 5.01 No references to any secret engine service (e.g.: Hashicorp Vault) - [x] TRG 5.01 Dependencies should be prefixed with the nameOverride and/or fullnameOverride properties - [x] TRG 5.01 Image tag is set to the `Chart.yaml` `appVersion` property - [x] TRG 5.01 must be deployable to any environment without overwriting default values with a simple helm install command - [x] TRG 5.01 dependencies have to be declared in Chart.yaml NOT requirements.yml

TRG 6 Released Helm Chart

TRG 7 Open Source Governance

Hints

Information Sharing

matbmoser commented 7 months ago

This TRG is missing: https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-4

saudkhan116 commented 7 months ago

This TRG is missing: https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-4

Hi @matbmoser, its added now.

Thanks for highlighting it.

matbmoser commented 7 months ago

With the Version v2.1.1 we include all the requirements from the TRGs additionally this also was implemented: https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/629

matbmoser commented 7 months ago

I will open this ticket so that it can be reviewed

hzierer commented 7 months ago

Thanks for all the preparatory work.

Can you please uncheck the 7.01 as the code of conduct file is outdated: #226

matbmoser commented 7 months ago

@hzierer thanks,

Good that you saw that, I thought that it did not changed respect to the previous release. Hope that is not the only thing we need to update. We will merge your PR now 👍🏻

I added the PR next to the 7.01 if you find something more please let us know.

Thank you 💯

matbmoser commented 6 months ago

231 was solved in release: https://github.com/eclipse-tractusx/digital-product-pass/releases/tag/v2.1.3

matbmoser commented 6 months ago

Charts updated here #235 to solve this issue #234

matbmoser commented 6 months ago

The latest charts we go are in v2.1.4

hzierer commented 6 months ago

almost everything good to go from System team side - one small issue with the documentation remains, which will be clarified tomorrow

matbmoser commented 6 months ago

Open discussion in issue: https://github.com/eclipse-tractusx/digital-product-pass/issues/236 to track progress. Closing this ticket.