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

Add shopware commands #74

Merged
merged 17 commits into from
Jan 21, 2025
Merged

Add shopware commands #74

merged 17 commits into from
Jan 21, 2025

Conversation

TrayserCassa
Copy link
Contributor

This pull request includes several significant changes and additions to the shopware-operator project, including new API types, controller updates, and modifications to existing structures. Below is a summary of the most important changes:

New API Types and Controllers:

  • Added new StoreExec and StoreSnapshot API types along with their respective specifications, statuses, and deep copy methods. (api/v1/exec.go, api/v1/execstatus.go, api/v1/storesnapshot_types.go, api/v1/zz_generated.deepcopy.go) [1] [2] [3] [4] [5] [6] [7] [8] [9]

Controller Updates:

  • Updated the main.go to include the new StoreExec and StoreSnapshot controllers, ensuring they are set up with the manager. (cmd/manager.go) [1] [2]

Modifications to Existing Structures:

  • Deprecated the ServiceAccountName field in StoreSpec and moved it to ContainerSpec to standardize the structure. (api/v1/store.go) [1] [2]
  • Renamed ShopCondition to StoreCondition to maintain consistency across the project. (api/v1/store_status.go) [1] [2]

Makefile and Project Configuration:

  • Updated the Makefile to change the build target from cmd/manager.go to cmd/main.go. (Makefile)
  • Added new resources for StoreExec and StoreSnapshot to the PROJECT and kustomization.yaml configuration files. (PROJECT, config/crd/kustomization.yaml) [1] [2]

These changes enhance the functionality and maintainability of the shopware-operator by introducing new API types, updating controllers, and refining the existing codebase.

@TrayserCassa TrayserCassa force-pushed the add-commands branch 2 times, most recently from 34a47ea to 0b85b59 Compare January 16, 2025 15:30
This is not needed because it will generate config files for kustomize
which is in the docker container not needed.
@TrayserCassa TrayserCassa self-assigned this Jan 20, 2025
@TrayserCassa TrayserCassa requested a review from tombojer January 20, 2025 14:53
@TrayserCassa TrayserCassa merged commit 151b391 into main Jan 21, 2025
4 checks passed
@TrayserCassa TrayserCassa deleted the add-commands branch January 21, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants