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

feat: enable seeding of operator information and test data #450

Merged
merged 11 commits into from
Oct 17, 2024

Conversation

Phil91
Copy link
Member

@Phil91 Phil91 commented Oct 9, 2024

Description

Set initial data

  • make operator company BPN, name, shortname and address overwriteable

Test data

  • defined a default test data set within configMap and enabled its seeding in a configurable manner
  • add configuration to add own config maps for test data seeding

Why

To be able to add custom test data and have the possibility to set the bpn for the operator via the configuration

Issue

Refs: #449 #447 eclipse-tractusx/tractus-x-umbrella#105

Corresponding Backend PR

eclipse-tractusx/portal-backend#1067

Checklist

  • I have performed a self-review of my changes
  • I have successfully tested my changes
  • I have added comments in the default values.yaml file with helm-docs syntax ('# -- ') if relevant for installation
  • I have commented my changes, particularly in hard-to-understand areas

@Phil91 Phil91 force-pushed the feature/449-seeding branch 2 times, most recently from 1c0e162 to ef49817 Compare October 9, 2024 13:28
@Phil91 Phil91 force-pushed the feature/449-seeding branch 2 times, most recently from e95a06c to f6ab0f1 Compare October 9, 2024 13:32
@Phil91 Phil91 marked this pull request as ready for review October 9, 2024 13:54
@Phil91 Phil91 requested a review from evegufy October 9, 2024 13:54
@Phil91 Phil91 added this to the Release 24.12 milestone Oct 9, 2024
@Phil91 Phil91 requested a review from evegufy October 15, 2024 20:35
charts/portal/values.yaml Outdated Show resolved Hide resolved
charts/portal/values.yaml Outdated Show resolved Hide resolved
charts/portal/values.yaml Outdated Show resolved Hide resolved
@Phil91 Phil91 requested a review from evegufy October 16, 2024 13:14
@evegufy evegufy changed the title feat(seeding): adjust configuration for seeding data feat: enable seeding of operator information (e.g. BPN) and test data Oct 16, 2024
@evegufy evegufy changed the title feat: enable seeding of operator information (e.g. BPN) and test data feat: enable seeding of operator information and test data Oct 16, 2024
Copy link
Contributor

@evegufy evegufy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested successfully.
I think the helm install is currently failing because updated the migration image isn't available yet, once eclipse-tractusx/portal-backend#1067 is merged and the image pushed the install should be successful.

@Phil91 Phil91 merged commit 7504373 into main Oct 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

2 participants