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

QG4 System Team checks #101

Closed
61 tasks done
FaGru3n opened this issue Apr 19, 2023 · 8 comments
Closed
61 tasks done

QG4 System Team checks #101

FaGru3n opened this issue Apr 19, 2023 · 8 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@FaGru3n
Copy link
Contributor

FaGru3n commented Apr 19, 2023

QG checks

Please keep this issue open until QG4 is concluded!

Product Name: BPDM
Product Owner: Tim Rybak
Dev SPOC: @nicoprow
Helm Chart Version Pool: 4.3.04.3.1
Helm Chart Version Gate: 3.3.03.3.1
App Version: 3.2.13.2.2
QG4 Approval: yes/no

TRG 1 Documentation

TRG 1.01

https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-1

  • an appropriate README.md file
TRG 1.02

https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-2

  • an appropriate INSTALL.md
TRG 1.03

https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-3

TRG 2 Git

TRG 2.01

https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-1

  • default branch is named main
TRG 2.03

https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-3

TRG 2.04

https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-4

  • a Leading product repository
  • repository name must be productname without prefix or suffix
  • should contain the release
  • references/urls to the product's other repositories
  • might contain product helm chart
  • README.md: contains the urls for the backend and frontend applications substitute Catena-X and refrence product links #109
  • Links to Repository
  • Does it reference all other repositories?
  • Does it contain the Product Helm chart?
TRG 2.05

https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-5

TRG 3 Kubernetes

TRG 3.02

https://eclipse-tractusx.github.io/docs/release/trg-3/trg-3-2

  • if data persistence is needed in Kubernetes the use of PersistentVolume and PersistentVolumeClaim resource

TRG 4 container

TRG 4.01

https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-1

  • All images must be tagged following semantic versioning
  • container is labeled correctly additionally to the latest tag
TRG 4.02

https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-2

  • must add a section to your top level README.md file, that contains information about the used base image
  • Java, Kotlin, ... if JVM based language use base image from Eclipse Temurin
TRG 4.03

https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-3

TRG 4.05

https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-5

  • released image must be present on GitHub Package registry or Dockerhub

TRG 5 Helm

TRG 5.01

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-01

  • Helm chart must be released
  • appropriate semantic versioning for version and appVersion has to be used in Chart.yaml
  • must not contain any environment specific values-xyz.yaml
  • values.yaml file must contain proper default values/placeholders
  • No hostname provided for ingress
  • Ingress is disabled
  • No references to any secret engine service (e.g.: Hashicorp Vault)
  • Dependencies should be prefixed with the nameOverride and/or fullnameOverride properties
  • Image tag is set to the Chart.yaml appVersion property Helm chart Version / App Version Missmatch #112
  • must be deployable to any environment without overwriting default values with a simple helm install command
  • dependencies have to be declared in Chart.yaml NOT requirements.yml
  • recommended use GitHub Chart Releaser Action
TRG 5.02

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-02

  • Helm chart location inside Git repository in /charts directory
  • chart file structure Licence headers #113
charts/ 
    chartNameA/
      Chart.yaml
      ... 
    chartNameB/
      Chart.yaml
      ...
AUTHORS.md 
DEPENDENCIES.md 
LICENCE 
README.md 
  • each file must contain the Apache 2.0 Licence
  • latest tag is not used in helm chart be default
TRG 5.04

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-04

  • CPU and memory limits and requests are properly set
TRG 5.06

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-06

  • every startup configuration aspect of your application must be configurable through the Helm chart (ingress class, tls, labels, annotations, database, secrets, persistence, env variables)
TRG 5.07

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-07

  • if dependencies are present in the Chart.yaml they are properly configured
TRG 5.08

https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-08

  • a product has a single deployable helm chart that contains all components (backend, frontend, etc.)
  • name of the Chart should be just the product-name without prefix or suffix
  • values file should contain all available variables (even from subcharts) with default values and comments about what they do
  • helm install command should successfully install the chart to any supported Kubernetes version cluster (without overwriting default values)
  • helm test runs without errors

Testing

  • installed and running on pre-prod without errors

Hints

#114

Information Sharing

@FaGru3n FaGru3n added the documentation Improvements or additions to documentation label Apr 19, 2023
@FaGru3n
Copy link
Contributor Author

FaGru3n commented Apr 19, 2023

Hi bpdm Team! 👋🏼

could you please provide us some more information, that we can check?

Product Name:   BPDM
Product Owner:  
Dev SPOC:  
Helm Chart Version Pool:  `4.3.0`
Helm Chart Version Gate:  `3.3.0`
App Version:  
QG4 Approval: yes/no

Thanks in advance.

@nicoprow
Copy link
Contributor

Product Name: BPDM
Product Owner: Tim Rybak
Dev SPOC: Nico Koprowski
Helm Chart Version Pool: 4.3.0
Helm Chart Version Gate: 3.3.0
App Version: 3.2.1
QG4 Approval: yes/no

@FaGru3n
Copy link
Contributor Author

FaGru3n commented Apr 27, 2023

Hi @nicoprow,

thanks for your feedback.

to avoid confusion on my side, you created tags within repo for each product componet?
So this leads to following git-tags?
Helm Chart Version Pool: 4.3.0 --> https://github.com/eclipse-tractusx/bpdm/tree/bpdm-pool-4.3.0
Helm Chart Version Gate: 3.3.0 --> https://github.com/eclipse-tractusx/bpdm/tree/bpdm-gate-3.3.0
App Version: 3.2.1

@FaGru3n
Copy link
Contributor Author

FaGru3n commented Apr 27, 2023

Hi @nicoprow,

created you some issue to checkout
#115
#113
#112
#110
#109
#108
#107
and a little hint
#114

@FaGru3n FaGru3n self-assigned this May 2, 2023
@FaGru3n
Copy link
Contributor Author

FaGru3n commented May 11, 2023

Hi @nicoprow,

seems only one task is open could maybe generate a new release and update the versions?
#112

@nicoprow
Copy link
Contributor

Due to changes towards charts and app the patch versions are now increased:

Helm Chart Version Pool: 4.3.1
Helm Chart Version Gate: 3.3.1
App Version: 3.2.2

@FaGru3n
Copy link
Contributor Author

FaGru3n commented May 12, 2023

Due to changes towards charts and app the patch versions are now increased:

Helm Chart Version Pool: 4.3.1 Helm Chart Version Gate: 3.3.1 App Version: 3.2.2

Pre-Prod charts seems running

@FaGru3n
Copy link
Contributor Author

FaGru3n commented May 12, 2023

updated chart reference and checked on pre-prod

@FaGru3n FaGru3n closed this as completed May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants