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

refactor: move all project to dedicated repo #1726

Merged
merged 4 commits into from
Sep 26, 2024
Merged

refactor: move all project to dedicated repo #1726

merged 4 commits into from
Sep 26, 2024

Conversation

flemzord
Copy link
Member

No description provided.

@flemzord flemzord requested a review from a team as a code owner September 26, 2024 16:10
Copy link

gitguardian bot commented Sep 26, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
4298688 Triggered Generic Password dd9f3b5 components/operator/.earthly/configuration/templates/postgres-secret.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Copy link
Contributor

coderabbitai bot commented Sep 26, 2024

Walkthrough

The pull request introduces significant changes by removing multiple jobs from the GitHub Actions workflow and deleting various configuration files related to Kubernetes resources and Docker. The workflow simplification results in the retention of only the TestsIntegration job. Additionally, several files related to Kubernetes secrets, Helm charts, and project configurations have been eliminated, streamlining the overall structure and reducing complexity.

Changes

File Path Change Summary
.github/workflows/main.yml Removed jobs: Tests, Extract, GoReleaser, Deploy; retained TestsIntegration.
Earthfile Added imports for flows, reconciliation, and wallets; replaced loop with explicit COPY commands.
components/operator/.dockerignore Removed .dockerignore file, eliminating ignore rules for Docker build.
components/operator/.earthly/configuration/.helmignore Deleted .helmignore, removing patterns for files to ignore when building packages.
components/operator/.earthly/configuration/Chart.yaml Removed Helm chart configuration for the "configuration" chart.
components/operator/.earthly/configuration/templates/*.yaml Deleted multiple Kubernetes Secret resources and settings configurations.
components/operator/.earthly/configuration/values.yaml Removed gateway.fallback configuration entry.
components/operator/.gitignore Deleted .gitignore file, which specified patterns for Git to ignore.
components/operator/Makefile Removed Makefile, which contained build, test, and deployment configurations.
components/operator/PROJECT Deleted project configuration for the "operatorv2".
components/operator/README.md Removed README file with documentation for the operator.
components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go Deleted autogenerated deep copy methods for various API objects.

Possibly related PRs

Suggested reviewers

  • gfyrag
  • Dav-14

Poem

🐇 In the meadow, changes bloom,
Jobs removed, like flowers' plume.
Secrets gone, the path is clear,
A simpler workflow, we hold dear.
Hopping forth with joy and cheer! 🌼


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Outside diff range and nitpick comments (1)
Earthfile (1)

41-43: LGTM! Explicit COPY commands added for new modules.

The new COPY commands for wallets, reconciliation, and orchestration OpenAPI specifications are consistent with the existing style and improve maintainability by making the process more explicit.

For consistency with other COPY commands, consider grouping these new commands with the existing ones. You can apply this diff:

     COPY (auth+openapi/openapi.yaml) /src/ee/auth/
     COPY (search+openapi/openapi.yaml) /src/ee/search/
     COPY (webhooks+openapi/openapi.yaml) /src/ee/webhooks/
-
-    COPY (wallets+openapi/openapi.yaml) /src/ee/wallets/
-    COPY (reconciliation+openapi/openapi.yaml) /src/ee/reconciliation/
-    COPY (orchestration+openapi/openapi.yaml) /src/ee/orchestration/
+    COPY (wallets+openapi/openapi.yaml) /src/ee/wallets/
+    COPY (reconciliation+openapi/openapi.yaml) /src/ee/reconciliation/
+    COPY (orchestration+openapi/openapi.yaml) /src/ee/orchestration/
📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between b37d1df and 91d7f42.

⛔ Files ignored due to path filters (163)
  • components/operator/config/crd/bases/formance.com_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_authclients.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_auths.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthos.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthosstreams.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokerconsumers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokertopics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_databases.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gatewayhttpapis.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gateways.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_ledgers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_orchestrations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_reconciliations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_resourcereferences.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_searches.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stacks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stargates.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomizeconfig.yaml is excluded by !**/*.yaml
  • components/operator/config/default/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_config_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/monitor.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_client_clusterrole.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_service.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/service_account.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_auth.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_authclient.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_broker.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconsumer.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_databaseconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_elasticsearchconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_gateway.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_ledger.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_opentelemetryconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_orchestration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_reconciliation.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_registriesconfigurations.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_resourcereference.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_search.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_searchbatchingconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_stack.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_temporalconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/webhook/manifests.yaml is excluded by !**/*.yaml
  • components/operator/docs.config.yaml is excluded by !**/*.yaml
  • components/operator/go.mod is excluded by !**/*.mod
  • components/operator/go.sum is excluded by !**/*.sum, !**/*.sum
  • components/operator/helm/crds/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_analytics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_authclients.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_auths.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthos.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthosstreams.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokerconsumers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokertopics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_databases.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gatewayhttpapis.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gateways.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_ledgers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_orchestrations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_payments.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_reconciliations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_resourcereferences.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_searches.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_settings.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stacks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stargates.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_versions.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_wallets.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_webhooks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/values.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • components/operator/helm/operator/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/deployment.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-manager-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-metrics-reader.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-proxy-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-manager-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-proxy-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_role_formance-leader-election-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_rolebinding_formance-leader-election-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_service_formance-controller-manager-metrics-service.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_serviceaccount_formance-controller-manager.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/licence-secret.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/values.yaml is excluded by !**/*.yaml
📒 Files selected for processing (76)
  • .github/workflows/main.yml (1 hunks)
  • Earthfile (2 hunks)
  • components/Earthfile (0 hunks)
  • components/operator/.dockerignore (0 hunks)
  • components/operator/.earthly/configuration/.helmignore (0 hunks)
  • components/operator/.earthly/configuration/Chart.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/settings.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/values.yaml (0 hunks)
  • components/operator/.earthly/k8s-versions.yaml (0 hunks)
  • components/operator/.gitignore (0 hunks)
  • components/operator/.goreleaser.yml (0 hunks)
  • components/operator/Earthfile (0 hunks)
  • components/operator/Makefile (0 hunks)
  • components/operator/PROJECT (0 hunks)
  • components/operator/README.md (0 hunks)
  • components/operator/api/formance.com/v1beta1/analytics_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/auth_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/authclient_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthos_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/broker_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/database_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gateway_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/groupversion_info.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/ledger_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/orchestration_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/payments_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/search_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/settings_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/shared.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stack_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stargate_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/versions_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/wallets_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/webhooks_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go (0 hunks)
  • components/operator/build.Dockerfile (0 hunks)
  • components/operator/cmd/main.go (0 hunks)
  • components/operator/crd-doc-templates/gv_details.tpl (0 hunks)
  • components/operator/crd-doc-templates/gv_list.tpl (0 hunks)
  • components/operator/crd-doc-templates/type.tpl (0 hunks)
  • components/operator/crd-doc-templates/type_members.tpl (0 hunks)
  • components/operator/docs/01-Requirements.md (0 hunks)
  • components/operator/docs/02-Installation.md (0 hunks)
  • components/operator/docs/03-Demo deployment.md (0 hunks)
  • components/operator/docs/04-Modules/01-Stack.md (0 hunks)
  • components/operator/docs/04-Modules/02-Gateway.md (0 hunks)
  • components/operator/docs/04-Modules/03-Ledger.md (0 hunks)
  • components/operator/docs/04-Modules/04-Payments.md (0 hunks)
  • components/operator/docs/04-Modules/05-Auth.md (0 hunks)
  • components/operator/docs/04-Modules/06-Orchestration.md (0 hunks)
  • components/operator/docs/04-Modules/07-Search.md (0 hunks)
  • components/operator/docs/04-Modules/08-Reconciliation.md (0 hunks)
  • components/operator/docs/04-Modules/09-Wallets.md (0 hunks)
  • components/operator/docs/04-Modules/10-Webhooks.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/02-Message broker.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/04-Temporal.md (0 hunks)
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md (0 hunks)
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md (0 hunks)
  • components/operator/docs/07-Upgrade/02-Database update.md (0 hunks)
  • components/operator/docs/08-Troubleshooting.md (0 hunks)
  • components/operator/docs/09-Configuration reference/01-Settings.md (0 hunks)
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md (0 hunks)
  • components/operator/hack/boilerplate.go.txt (0 hunks)
  • components/operator/helm/crds/.helmignore (0 hunks)
  • components/operator/helm/crds/README.md (0 hunks)
💤 Files not reviewed due to no reviewable changes (74)
  • components/Earthfile
  • components/operator/.dockerignore
  • components/operator/.earthly/configuration/.helmignore
  • components/operator/.earthly/configuration/Chart.yaml
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml
  • components/operator/.earthly/configuration/templates/settings.yaml
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml
  • components/operator/.earthly/configuration/values.yaml
  • components/operator/.earthly/k8s-versions.yaml
  • components/operator/.gitignore
  • components/operator/.goreleaser.yml
  • components/operator/Earthfile
  • components/operator/Makefile
  • components/operator/PROJECT
  • components/operator/README.md
  • components/operator/api/formance.com/v1beta1/analytics_types.go
  • components/operator/api/formance.com/v1beta1/auth_types.go
  • components/operator/api/formance.com/v1beta1/authclient_types.go
  • components/operator/api/formance.com/v1beta1/benthos_types.go
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go
  • components/operator/api/formance.com/v1beta1/broker_types.go
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go
  • components/operator/api/formance.com/v1beta1/database_types.go
  • components/operator/api/formance.com/v1beta1/gateway_types.go
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go
  • components/operator/api/formance.com/v1beta1/groupversion_info.go
  • components/operator/api/formance.com/v1beta1/ledger_types.go
  • components/operator/api/formance.com/v1beta1/orchestration_types.go
  • components/operator/api/formance.com/v1beta1/payments_types.go
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go
  • components/operator/api/formance.com/v1beta1/search_types.go
  • components/operator/api/formance.com/v1beta1/settings_types.go
  • components/operator/api/formance.com/v1beta1/shared.go
  • components/operator/api/formance.com/v1beta1/stack_types.go
  • components/operator/api/formance.com/v1beta1/stargate_types.go
  • components/operator/api/formance.com/v1beta1/versions_types.go
  • components/operator/api/formance.com/v1beta1/wallets_types.go
  • components/operator/api/formance.com/v1beta1/webhooks_types.go
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go
  • components/operator/build.Dockerfile
  • components/operator/cmd/main.go
  • components/operator/crd-doc-templates/gv_details.tpl
  • components/operator/crd-doc-templates/gv_list.tpl
  • components/operator/crd-doc-templates/type.tpl
  • components/operator/crd-doc-templates/type_members.tpl
  • components/operator/docs/01-Requirements.md
  • components/operator/docs/02-Installation.md
  • components/operator/docs/03-Demo deployment.md
  • components/operator/docs/04-Modules/01-Stack.md
  • components/operator/docs/04-Modules/02-Gateway.md
  • components/operator/docs/04-Modules/03-Ledger.md
  • components/operator/docs/04-Modules/04-Payments.md
  • components/operator/docs/04-Modules/05-Auth.md
  • components/operator/docs/04-Modules/06-Orchestration.md
  • components/operator/docs/04-Modules/07-Search.md
  • components/operator/docs/04-Modules/08-Reconciliation.md
  • components/operator/docs/04-Modules/09-Wallets.md
  • components/operator/docs/04-Modules/10-Webhooks.md
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md
  • components/operator/docs/05-Infrastructure services/02-Message broker.md
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md
  • components/operator/docs/05-Infrastructure services/04-Temporal.md
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md
  • components/operator/docs/07-Upgrade/02-Database update.md
  • components/operator/docs/08-Troubleshooting.md
  • components/operator/docs/09-Configuration reference/01-Settings.md
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md
  • components/operator/hack/boilerplate.go.txt
  • components/operator/helm/crds/.helmignore
  • components/operator/helm/crds/README.md
🔇 Additional comments (6)
.github/workflows/main.yml (4)

Line range hint 1-61: Verify the completeness of the CI/CD pipeline after job removals.

The workflow has been significantly simplified by removing several jobs (Tests, Extract, GoReleaser, and Deploy). While this may streamline the process, it's crucial to ensure that all necessary checks and deployments are still being performed, either in this workflow or in separate workflows.

Please confirm that:

  1. All required tests are still being executed.
  2. The release process is handled appropriately.
  3. Deployment procedures are covered, if needed.

If these aspects are now managed in separate workflows or repositories, consider adding comments in this file to reference them for clarity.


61-61: LGTM: Proper use of secrets for API key.

The indentation of the SPEAKEASY_API_KEY secret looks correct. Using secrets for API keys is a good security practice.


Line range hint 1-61: Address potential gaps in CI/CD pipeline due to removed jobs.

The removal of several key jobs (Tests, Extract, GoReleaser, and Deploy) from this workflow could lead to gaps in your CI/CD pipeline. Please ensure that:

  1. Test Coverage: If the removed "Tests" job was responsible for unit tests, make sure they are now run in another workflow or as part of the remaining jobs.

  2. Service Extraction: If the "Extract" job was crucial for identifying services, ensure this functionality is handled elsewhere if still needed.

  3. Release Process: The removal of the "GoReleaser" job might impact your release process. Verify that releases are managed appropriately in another workflow or system.

  4. Deployment: With the "Deploy" job removed, confirm that your deployment process is covered in another workflow or through a different mechanism.

To check for potential gaps:

#!/bin/bash
# Search for workflows that might handle the removed functionalities
echo "Workflows potentially handling tests:"
fd -e yml -e yaml . .github/workflows | xargs grep -n "run:.*test"
echo "Workflows potentially handling releases:"
fd -e yml -e yaml . .github/workflows | xargs grep -n "goreleaser"
echo "Workflows potentially handling deployments:"
fd -e yml -e yaml . .github/workflows | xargs grep -n "deploy"

Line range hint 17-61: Ensure comprehensive test coverage and review custom runner usage.

  1. Unit Tests: The removal of the "Tests" job suggests that unit tests might no longer be running in this workflow. If they're not covered elsewhere, consider reintroducing them to maintain code quality.

  2. Custom Runner: Both jobs use a custom runner "formance-runner". Ensure this runner is properly maintained and has all necessary dependencies for the jobs it's running.

  3. Earthly Usage: The workflow heavily relies on Earthly. Make sure all team members are familiar with this tool and that it's consistently used across your projects.

To confirm the test coverage:

Earthfile (2)

Line range hint 1-1: Overall, the changes look good and align with the PR objective.

The modifications to the Earthfile, including new imports and updates to the build-final-spec section, are consistent with the goal of refactoring the project structure. The changes appear to be well-implemented and maintain the existing coding style.

Key points:

  1. New imports for flows, reconciliation, and wallets have been added correctly.
  2. The build-final-spec section has been updated with explicit COPY commands for the new modules.
  3. Some BUILD commands have been removed, as mentioned in the AI summary.

Please ensure that all verification steps mentioned in the previous comments are completed to confirm that these changes don't negatively impact the build and test processes.


Line range hint 1-1: Verify the impact of removed BUILD commands.

The AI summary mentions that BUILD commands for ./components+run and ./ee+run have been removed from the tests, pre-commit, and tidy sections. While these changes align with the PR objective, it's important to ensure they don't negatively impact the build and test processes.

Please run the following script to verify the changes and their impact:

#!/bin/bash
# Description: Verify the removal of BUILD commands and their impact

# Test 1: Check for the presence of removed BUILD commands
echo "Checking for removed BUILD commands:"
rg --type-add 'earthfile:*.Earthfile' --type earthfile 'BUILD.*\./components\+run' Earthfile
rg --type-add 'earthfile:*.Earthfile' --type earthfile 'BUILD.*\./ee\+run' Earthfile

# Test 2: Verify the content of tests, pre-commit, and tidy targets
echo "Verifying content of affected targets:"
rg --type-add 'earthfile:*.Earthfile' --type earthfile -A 10 '^tests:' Earthfile
rg --type-add 'earthfile:*.Earthfile' --type earthfile -A 10 '^pre-commit:' Earthfile
rg --type-add 'earthfile:*.Earthfile' --type earthfile -A 10 '^tidy:' Earthfile

Please review the output of this script to ensure that:

  1. The removed BUILD commands are no longer present.
  2. The tests, pre-commit, and tidy targets still contain the necessary steps to maintain the build and test processes.

Comment on lines +12 to +14
IMPORT github.com/formancehq/flows:main AS orchestration
IMPORT github.com/formancehq/reconciliation:main AS reconciliation
IMPORT github.com/formancehq/wallets:main AS wallets
Copy link
Contributor

Choose a reason for hiding this comment

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

💡 Codebase verification

Unintended Removal of legacy Import Detected

The import statement IMPORT github.com/formancehq/legacy:main AS legacy has been unexpectedly removed from the Earthfile. Please verify if this removal was intentional to ensure it does not impact dependent components.

🔗 Analysis chain

LGTM! New imports added consistently.

The new import statements for flows, reconciliation, and wallets are consistent with the existing import style and align with the PR objective of moving projects to dedicated repositories.

To ensure all necessary imports are present, run the following script:

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify all required imports are present in the Earthfile

# Test: Check for the presence of all expected imports
rg --type-add 'earthfile:*.Earthfile' --type earthfile '^IMPORT github.com/formancehq/' Earthfile

Length of output: 1281


Script:

#!/bin/bash
# Description: Verify the presence of new import statements and check for unintended removals in the Earthfile

# Define the expected import statements
EXPECTED_IMPORTS=(
  'IMPORT github.com/formancehq/flows:main AS orchestration'
  'IMPORT github.com/formancehq/reconciliation:main AS reconciliation'
  'IMPORT github.com/formancehq/wallets:main AS wallets'
)

# Check for each expected import
for IMPORT in "${EXPECTED_IMPORTS[@]}"; do
  if ! rg --type-add 'earthfile:*.Earthfile' --type earthfile "^${IMPORT}$" Earthfile > /dev/null; then
    echo "Missing import: ${IMPORT}"
  fi
done

# Optionally, check that no unexpected imports have been removed
# This requires knowing the previous state or having a list of imports that should remain
# Example: Ensure that existing imports other than the new ones are still present
EXISTING_IMPORTS=(
  'IMPORT github.com/formancehq/legacy:main AS legacy'
  # Add other existing imports as needed
)

for IMPORT in "${EXISTING_IMPORTS[@]}"; do
  if ! rg --type-add 'earthfile:*.Earthfile' --type earthfile "^${IMPORT}$" Earthfile > /dev/null; then
    echo "Unexpected removal of import: ${IMPORT}"
  fi
done

Length of output: 1124

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (3)
.github/workflows/main.yml (2)

Line range hint 1-61: Overall workflow changes align with PR objective, but require careful verification.

The significant simplification of the workflow aligns well with the PR objective of moving the project to a dedicated repo. However, to ensure a smooth transition and maintain the integrity of the CI/CD process, please address the following points:

  1. Confirm that all necessary jobs (Tests, Extract, GoReleaser, Deploy) have been appropriately relocated to the new dedicated repo's workflow.
  2. Verify that the simplified workflow in this repo is sufficient for its new role in the project structure.
  3. Ensure that the custom runner "formance-runner" is still appropriate for both jobs in the new context.
  4. Review the integration tests to confirm they are still relevant and properly utilize the new SPEAKEASY_API_KEY.
  5. Double-check that no critical CI/CD steps have been inadvertently omitted in this transition.

These verifications will help maintain the project's quality and integrity while adapting to its new structure.

Consider documenting the new project structure and the role of this repository in the project's README or a separate architecture document. This will help future contributors understand the purpose of this simplified workflow in the context of the larger project.


Incomplete Removal of Workflow Jobs

The Extract and GoReleaser jobs are still present in .github/workflows/releases.yml. This suggests that these jobs have not been fully relocated to the new dedicated repository.

Please ensure that:

  • The Extract and GoReleaser jobs are moved entirely to the dedicated repository's workflow.
  • They are removed from all workflow files in the current repository to prevent redundancy and potential conflicts.
🔗 Analysis chain

Line range hint 1-61: Workflow structure aligns with PR objective, but verify completeness.

The significant simplification of the workflow, including the removal of Tests, Extract, GoReleaser, and Deploy jobs, aligns with the PR objective of moving the project to a dedicated repo. This streamlined workflow focuses on basic checks ("Dirty") and integration tests, which could be appropriate if this is no longer the main project repository.

However, please confirm that:

  1. The removed jobs have been appropriately relocated to the new dedicated repo's workflow.
  2. This simplified workflow is sufficient for this repository's new role in the project structure.

To ensure all necessary jobs have been moved, please run the following script:

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Check for the presence of removed jobs in other workflow files

# Test: Search for removed job names in other workflow files
echo "Searching for removed jobs in other workflow files:"
rg --type yaml 'Tests:|Extract:|GoReleaser:|Deploy:' .github/workflows

Length of output: 216


Script:

#!/bin/bash
# Description: Check for the presence of removed jobs in other workflow files

# Search for removed job names in other workflow files
echo "Searching for removed jobs in other workflow files:"
rg --type yaml "Tests:|Extract:|GoReleaser:|Deploy:" .github/workflows

Length of output: 272

Earthfile (1)

12-14: New imports added correctly, consider version pinning.

The new imports for orchestration, reconciliation, and wallets align well with the PR objective of moving the project to a dedicated repo. The aliases used are clear and descriptive.

Consider pinning these imports to specific versions or tags instead of using the main branch. This can help ensure build stability and reproducibility. For example:

-IMPORT github.com/formancehq/flows:main AS orchestration
-IMPORT github.com/formancehq/reconciliation:main AS reconciliation
-IMPORT github.com/formancehq/wallets:main AS wallets
+IMPORT github.com/formancehq/flows:v1.0.0 AS orchestration
+IMPORT github.com/formancehq/reconciliation:v1.0.0 AS reconciliation
+IMPORT github.com/formancehq/wallets:v1.0.0 AS wallets

Replace v1.0.0 with the appropriate version tags for each repository.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between b37d1df and 91d7f42.

⛔ Files ignored due to path filters (163)
  • components/operator/config/crd/bases/formance.com_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_authclients.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_auths.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthos.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthosstreams.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokerconsumers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokertopics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_databases.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gatewayhttpapis.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gateways.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_ledgers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_orchestrations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_reconciliations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_resourcereferences.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_searches.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stacks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stargates.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomizeconfig.yaml is excluded by !**/*.yaml
  • components/operator/config/default/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_config_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/monitor.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_client_clusterrole.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_service.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/service_account.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_auth.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_authclient.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_broker.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconsumer.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_databaseconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_elasticsearchconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_gateway.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_ledger.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_opentelemetryconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_orchestration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_reconciliation.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_registriesconfigurations.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_resourcereference.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_search.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_searchbatchingconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_stack.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_temporalconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/webhook/manifests.yaml is excluded by !**/*.yaml
  • components/operator/docs.config.yaml is excluded by !**/*.yaml
  • components/operator/go.mod is excluded by !**/*.mod
  • components/operator/go.sum is excluded by !**/*.sum, !**/*.sum
  • components/operator/helm/crds/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_analytics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_authclients.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_auths.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthos.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthosstreams.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokerconsumers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokertopics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_databases.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gatewayhttpapis.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gateways.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_ledgers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_orchestrations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_payments.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_reconciliations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_resourcereferences.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_searches.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_settings.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stacks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stargates.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_versions.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_wallets.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_webhooks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/values.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • components/operator/helm/operator/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/deployment.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-manager-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-metrics-reader.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-proxy-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-manager-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-proxy-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_role_formance-leader-election-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_rolebinding_formance-leader-election-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_service_formance-controller-manager-metrics-service.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_serviceaccount_formance-controller-manager.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/licence-secret.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/values.yaml is excluded by !**/*.yaml
📒 Files selected for processing (76)
  • .github/workflows/main.yml (1 hunks)
  • Earthfile (2 hunks)
  • components/Earthfile (0 hunks)
  • components/operator/.dockerignore (0 hunks)
  • components/operator/.earthly/configuration/.helmignore (0 hunks)
  • components/operator/.earthly/configuration/Chart.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/settings.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/values.yaml (0 hunks)
  • components/operator/.earthly/k8s-versions.yaml (0 hunks)
  • components/operator/.gitignore (0 hunks)
  • components/operator/.goreleaser.yml (0 hunks)
  • components/operator/Earthfile (0 hunks)
  • components/operator/Makefile (0 hunks)
  • components/operator/PROJECT (0 hunks)
  • components/operator/README.md (0 hunks)
  • components/operator/api/formance.com/v1beta1/analytics_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/auth_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/authclient_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthos_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/broker_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/database_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gateway_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/groupversion_info.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/ledger_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/orchestration_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/payments_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/search_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/settings_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/shared.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stack_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stargate_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/versions_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/wallets_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/webhooks_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go (0 hunks)
  • components/operator/build.Dockerfile (0 hunks)
  • components/operator/cmd/main.go (0 hunks)
  • components/operator/crd-doc-templates/gv_details.tpl (0 hunks)
  • components/operator/crd-doc-templates/gv_list.tpl (0 hunks)
  • components/operator/crd-doc-templates/type.tpl (0 hunks)
  • components/operator/crd-doc-templates/type_members.tpl (0 hunks)
  • components/operator/docs/01-Requirements.md (0 hunks)
  • components/operator/docs/02-Installation.md (0 hunks)
  • components/operator/docs/03-Demo deployment.md (0 hunks)
  • components/operator/docs/04-Modules/01-Stack.md (0 hunks)
  • components/operator/docs/04-Modules/02-Gateway.md (0 hunks)
  • components/operator/docs/04-Modules/03-Ledger.md (0 hunks)
  • components/operator/docs/04-Modules/04-Payments.md (0 hunks)
  • components/operator/docs/04-Modules/05-Auth.md (0 hunks)
  • components/operator/docs/04-Modules/06-Orchestration.md (0 hunks)
  • components/operator/docs/04-Modules/07-Search.md (0 hunks)
  • components/operator/docs/04-Modules/08-Reconciliation.md (0 hunks)
  • components/operator/docs/04-Modules/09-Wallets.md (0 hunks)
  • components/operator/docs/04-Modules/10-Webhooks.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/02-Message broker.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/04-Temporal.md (0 hunks)
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md (0 hunks)
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md (0 hunks)
  • components/operator/docs/07-Upgrade/02-Database update.md (0 hunks)
  • components/operator/docs/08-Troubleshooting.md (0 hunks)
  • components/operator/docs/09-Configuration reference/01-Settings.md (0 hunks)
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md (0 hunks)
  • components/operator/hack/boilerplate.go.txt (0 hunks)
  • components/operator/helm/crds/.helmignore (0 hunks)
  • components/operator/helm/crds/README.md (0 hunks)
💤 Files not reviewed due to no reviewable changes (74)
  • components/Earthfile
  • components/operator/.dockerignore
  • components/operator/.earthly/configuration/.helmignore
  • components/operator/.earthly/configuration/Chart.yaml
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml
  • components/operator/.earthly/configuration/templates/settings.yaml
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml
  • components/operator/.earthly/configuration/values.yaml
  • components/operator/.earthly/k8s-versions.yaml
  • components/operator/.gitignore
  • components/operator/.goreleaser.yml
  • components/operator/Earthfile
  • components/operator/Makefile
  • components/operator/PROJECT
  • components/operator/README.md
  • components/operator/api/formance.com/v1beta1/analytics_types.go
  • components/operator/api/formance.com/v1beta1/auth_types.go
  • components/operator/api/formance.com/v1beta1/authclient_types.go
  • components/operator/api/formance.com/v1beta1/benthos_types.go
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go
  • components/operator/api/formance.com/v1beta1/broker_types.go
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go
  • components/operator/api/formance.com/v1beta1/database_types.go
  • components/operator/api/formance.com/v1beta1/gateway_types.go
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go
  • components/operator/api/formance.com/v1beta1/groupversion_info.go
  • components/operator/api/formance.com/v1beta1/ledger_types.go
  • components/operator/api/formance.com/v1beta1/orchestration_types.go
  • components/operator/api/formance.com/v1beta1/payments_types.go
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go
  • components/operator/api/formance.com/v1beta1/search_types.go
  • components/operator/api/formance.com/v1beta1/settings_types.go
  • components/operator/api/formance.com/v1beta1/shared.go
  • components/operator/api/formance.com/v1beta1/stack_types.go
  • components/operator/api/formance.com/v1beta1/stargate_types.go
  • components/operator/api/formance.com/v1beta1/versions_types.go
  • components/operator/api/formance.com/v1beta1/wallets_types.go
  • components/operator/api/formance.com/v1beta1/webhooks_types.go
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go
  • components/operator/build.Dockerfile
  • components/operator/cmd/main.go
  • components/operator/crd-doc-templates/gv_details.tpl
  • components/operator/crd-doc-templates/gv_list.tpl
  • components/operator/crd-doc-templates/type.tpl
  • components/operator/crd-doc-templates/type_members.tpl
  • components/operator/docs/01-Requirements.md
  • components/operator/docs/02-Installation.md
  • components/operator/docs/03-Demo deployment.md
  • components/operator/docs/04-Modules/01-Stack.md
  • components/operator/docs/04-Modules/02-Gateway.md
  • components/operator/docs/04-Modules/03-Ledger.md
  • components/operator/docs/04-Modules/04-Payments.md
  • components/operator/docs/04-Modules/05-Auth.md
  • components/operator/docs/04-Modules/06-Orchestration.md
  • components/operator/docs/04-Modules/07-Search.md
  • components/operator/docs/04-Modules/08-Reconciliation.md
  • components/operator/docs/04-Modules/09-Wallets.md
  • components/operator/docs/04-Modules/10-Webhooks.md
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md
  • components/operator/docs/05-Infrastructure services/02-Message broker.md
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md
  • components/operator/docs/05-Infrastructure services/04-Temporal.md
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md
  • components/operator/docs/07-Upgrade/02-Database update.md
  • components/operator/docs/08-Troubleshooting.md
  • components/operator/docs/09-Configuration reference/01-Settings.md
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md
  • components/operator/hack/boilerplate.go.txt
  • components/operator/helm/crds/.helmignore
  • components/operator/helm/crds/README.md
🔇 Additional comments (4)
.github/workflows/main.yml (2)

Line range hint 17-36: Verify the appropriateness of the "Dirty" job in the new context.

The "Dirty" job performs basic checks using a custom runner and Earthly for pre-commit checks, which is good for maintaining code quality. However, consider the following:

  1. Verify if the custom runner "formance-runner" is still appropriate for the new repo structure.
  2. Ensure that the Earthly setup and pre-commit checks are still relevant for the project in its new location.

To check the custom runner configuration, run:

#!/bin/bash
# Description: Check for custom runner configuration

# Test: Search for runner configuration files
echo "Searching for runner configuration files:"
fd -t f 'config.yml' .github/

Line range hint 38-61: Verify the changes in the "TestsIntegration" job and the new secret.

The "TestsIntegration" job has been updated to include a new secret SPEAKEASY_API_KEY. This change suggests that the integration tests now require this API key. Please ensure that:

  1. The SPEAKEASY_API_KEY secret is properly set in the repository secrets.
  2. The integration tests have been updated to use this new API key correctly.
  3. The use of this API key doesn't introduce any security risks or unintended dependencies.

Additionally, verify that the integration tests are still appropriate for the project in its new context after the move to a dedicated repo.

To check if the secret is properly set, run:

Earthfile (2)

41-43: OpenAPI spec copying simplified and aligned with new imports.

The new COPY commands for wallets, reconciliation, and orchestration OpenAPI specs are consistent with the newly added imports. This change simplifies the build process by explicitly copying each spec file instead of using a loop.

The explicit copying makes the build process more transparent and easier to maintain. Good job on this simplification.


Line range hint 1-1: Clarification needed on removed BUILD commands.

The AI summary mentions that BUILD commands for components and ee have been removed from the tests, pre-commit, and tidy sections. However, these changes are not visible in the provided code.

Could you please provide more information about these removals? Specifically:

  1. Which exact BUILD commands were removed?
  2. What was the rationale behind removing these commands?
  3. How will the testing and quality assurance processes be affected by these removals?
  4. Are there any replacement processes or commands that will ensure the same level of testing and code quality?

To help verify these changes, please run the following script:

This will help us understand the exact changes made and their potential impact on the project's build and test processes.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (1)
Earthfile (1)

41-43: Approve changes with a suggestion for improvement.

The explicit COPY commands for each module's OpenAPI specification improve clarity and align well with the new import structure. This change simplifies the build process and makes it easier to understand.

To further improve maintainability, consider using a variable or a loop for the destination path ("/src/ee/") if more modules are expected to be added in the future. This could help reduce repetition and make future additions easier.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between b37d1df and 91d7f42.

⛔ Files ignored due to path filters (163)
  • components/operator/config/crd/bases/formance.com_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_authclients.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_auths.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthos.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_benthosstreams.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokerconsumers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_brokertopics.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_databases.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gatewayhttpapis.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_gateways.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_ledgers.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_orchestrations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_reconciliations.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_resourcereferences.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_searches.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stacks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_stargates.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/bases/formance.com_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/crd/kustomizeconfig.yaml is excluded by !**/*.yaml
  • components/operator/config/default/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/default/manager_config_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager.yaml is excluded by !**/*.yaml
  • components/operator/config/manager/manager_auth_proxy_patch.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/prometheus/monitor.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_client_clusterrole.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_proxy_service.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/auth_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/authclient_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthos_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/benthosstream_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/brokertopic_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/database_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_analytics_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_broker_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_brokerconsumer_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_resourcereference_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_settings_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gateway_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/gatewayhttpapi_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/leader_election_role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/ledger_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/orchestration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/payments_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/reconciliation_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/role_binding.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/search_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/service_account.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_configuration_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack.formance.com_versions_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stack_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/stargate_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/wallet_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_editor_role.yaml is excluded by !**/*.yaml
  • components/operator/config/rbac/webhooks_viewer_role.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_analytics.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_auth.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_authclient.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_broker.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_brokerconsumer.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_databaseconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_elasticsearchconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_gateway.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_ledger.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_opentelemetryconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_orchestration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_payments.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_reconciliation.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_registriesconfigurations.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_resourcereference.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_search.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_searchbatchingconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_settings.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_stack.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_temporalconfiguration.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_versions.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_wallets.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/formance.com_v1beta1_webhooks.yaml is excluded by !**/*.yaml
  • components/operator/config/samples/kustomization.yaml is excluded by !**/*.yaml
  • components/operator/config/webhook/manifests.yaml is excluded by !**/*.yaml
  • components/operator/docs.config.yaml is excluded by !**/*.yaml
  • components/operator/go.mod is excluded by !**/*.mod
  • components/operator/go.sum is excluded by !**/*.sum, !**/*.sum
  • components/operator/helm/crds/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_analytics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_authclients.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_auths.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthos.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_benthosstreams.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokerconsumers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_brokertopics.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_databases.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gatewayhttpapis.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_gateways.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_ledgers.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_orchestrations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_payments.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_reconciliations.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_resourcereferences.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_searches.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_settings.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stacks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_stargates.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_versions.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_wallets.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/templates/crds/apiextensions.k8s.io_v1_customresourcedefinition_webhooks.formance.com.yaml is excluded by !**/*.yaml
  • components/operator/helm/crds/values.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • components/operator/helm/operator/Chart.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/deployment.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-manager-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-metrics-reader.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrole_formance-proxy-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-manager-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_clusterrolebinding_formance-proxy-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_role_formance-leader-election-role.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/rbac.authorization.k8s.io_v1_rolebinding_formance-leader-election-rolebinding.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_service_formance-controller-manager-metrics-service.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/gen/v1_serviceaccount_formance-controller-manager.yaml is excluded by !**/gen/**, !**/*.yaml, !**/gen/**
  • components/operator/helm/operator/templates/licence-secret.yaml is excluded by !**/*.yaml
  • components/operator/helm/operator/values.yaml is excluded by !**/*.yaml
📒 Files selected for processing (76)
  • .github/workflows/main.yml (1 hunks)
  • Earthfile (2 hunks)
  • components/Earthfile (0 hunks)
  • components/operator/.dockerignore (0 hunks)
  • components/operator/.earthly/configuration/.helmignore (0 hunks)
  • components/operator/.earthly/configuration/Chart.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/settings.yaml (0 hunks)
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml (0 hunks)
  • components/operator/.earthly/configuration/values.yaml (0 hunks)
  • components/operator/.earthly/k8s-versions.yaml (0 hunks)
  • components/operator/.gitignore (0 hunks)
  • components/operator/.goreleaser.yml (0 hunks)
  • components/operator/Earthfile (0 hunks)
  • components/operator/Makefile (0 hunks)
  • components/operator/PROJECT (0 hunks)
  • components/operator/README.md (0 hunks)
  • components/operator/api/formance.com/v1beta1/analytics_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/auth_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/authclient_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthos_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/broker_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/database_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gateway_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/groupversion_info.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/ledger_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/orchestration_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/payments_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/search_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/settings_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/shared.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stack_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/stargate_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/versions_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/wallets_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/webhooks_types.go (0 hunks)
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go (0 hunks)
  • components/operator/build.Dockerfile (0 hunks)
  • components/operator/cmd/main.go (0 hunks)
  • components/operator/crd-doc-templates/gv_details.tpl (0 hunks)
  • components/operator/crd-doc-templates/gv_list.tpl (0 hunks)
  • components/operator/crd-doc-templates/type.tpl (0 hunks)
  • components/operator/crd-doc-templates/type_members.tpl (0 hunks)
  • components/operator/docs/01-Requirements.md (0 hunks)
  • components/operator/docs/02-Installation.md (0 hunks)
  • components/operator/docs/03-Demo deployment.md (0 hunks)
  • components/operator/docs/04-Modules/01-Stack.md (0 hunks)
  • components/operator/docs/04-Modules/02-Gateway.md (0 hunks)
  • components/operator/docs/04-Modules/03-Ledger.md (0 hunks)
  • components/operator/docs/04-Modules/04-Payments.md (0 hunks)
  • components/operator/docs/04-Modules/05-Auth.md (0 hunks)
  • components/operator/docs/04-Modules/06-Orchestration.md (0 hunks)
  • components/operator/docs/04-Modules/07-Search.md (0 hunks)
  • components/operator/docs/04-Modules/08-Reconciliation.md (0 hunks)
  • components/operator/docs/04-Modules/09-Wallets.md (0 hunks)
  • components/operator/docs/04-Modules/10-Webhooks.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/02-Message broker.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md (0 hunks)
  • components/operator/docs/05-Infrastructure services/04-Temporal.md (0 hunks)
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md (0 hunks)
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md (0 hunks)
  • components/operator/docs/07-Upgrade/02-Database update.md (0 hunks)
  • components/operator/docs/08-Troubleshooting.md (0 hunks)
  • components/operator/docs/09-Configuration reference/01-Settings.md (0 hunks)
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md (0 hunks)
  • components/operator/hack/boilerplate.go.txt (0 hunks)
  • components/operator/helm/crds/.helmignore (0 hunks)
  • components/operator/helm/crds/README.md (0 hunks)
💤 Files not reviewed due to no reviewable changes (74)
  • components/Earthfile
  • components/operator/.dockerignore
  • components/operator/.earthly/configuration/.helmignore
  • components/operator/.earthly/configuration/Chart.yaml
  • components/operator/.earthly/configuration/templates/elasticsearch-secret.yaml
  • components/operator/.earthly/configuration/templates/postgres-secret.yaml
  • components/operator/.earthly/configuration/templates/settings.yaml
  • components/operator/.earthly/configuration/templates/temporal-secret.yaml
  • components/operator/.earthly/configuration/values.yaml
  • components/operator/.earthly/k8s-versions.yaml
  • components/operator/.gitignore
  • components/operator/.goreleaser.yml
  • components/operator/Earthfile
  • components/operator/Makefile
  • components/operator/PROJECT
  • components/operator/README.md
  • components/operator/api/formance.com/v1beta1/analytics_types.go
  • components/operator/api/formance.com/v1beta1/auth_types.go
  • components/operator/api/formance.com/v1beta1/authclient_types.go
  • components/operator/api/formance.com/v1beta1/benthos_types.go
  • components/operator/api/formance.com/v1beta1/benthosstream_types.go
  • components/operator/api/formance.com/v1beta1/broker_types.go
  • components/operator/api/formance.com/v1beta1/brokerconsumer_types.go
  • components/operator/api/formance.com/v1beta1/brokertopic_types.go
  • components/operator/api/formance.com/v1beta1/database_types.go
  • components/operator/api/formance.com/v1beta1/gateway_types.go
  • components/operator/api/formance.com/v1beta1/gatewayhttpapi_types.go
  • components/operator/api/formance.com/v1beta1/groupversion_info.go
  • components/operator/api/formance.com/v1beta1/ledger_types.go
  • components/operator/api/formance.com/v1beta1/orchestration_types.go
  • components/operator/api/formance.com/v1beta1/payments_types.go
  • components/operator/api/formance.com/v1beta1/reconciliation_types.go
  • components/operator/api/formance.com/v1beta1/resourcereference_types.go
  • components/operator/api/formance.com/v1beta1/search_types.go
  • components/operator/api/formance.com/v1beta1/settings_types.go
  • components/operator/api/formance.com/v1beta1/shared.go
  • components/operator/api/formance.com/v1beta1/stack_types.go
  • components/operator/api/formance.com/v1beta1/stargate_types.go
  • components/operator/api/formance.com/v1beta1/versions_types.go
  • components/operator/api/formance.com/v1beta1/wallets_types.go
  • components/operator/api/formance.com/v1beta1/webhooks_types.go
  • components/operator/api/formance.com/v1beta1/zz_generated.deepcopy.go
  • components/operator/build.Dockerfile
  • components/operator/cmd/main.go
  • components/operator/crd-doc-templates/gv_details.tpl
  • components/operator/crd-doc-templates/gv_list.tpl
  • components/operator/crd-doc-templates/type.tpl
  • components/operator/crd-doc-templates/type_members.tpl
  • components/operator/docs/01-Requirements.md
  • components/operator/docs/02-Installation.md
  • components/operator/docs/03-Demo deployment.md
  • components/operator/docs/04-Modules/01-Stack.md
  • components/operator/docs/04-Modules/02-Gateway.md
  • components/operator/docs/04-Modules/03-Ledger.md
  • components/operator/docs/04-Modules/04-Payments.md
  • components/operator/docs/04-Modules/05-Auth.md
  • components/operator/docs/04-Modules/06-Orchestration.md
  • components/operator/docs/04-Modules/07-Search.md
  • components/operator/docs/04-Modules/08-Reconciliation.md
  • components/operator/docs/04-Modules/09-Wallets.md
  • components/operator/docs/04-Modules/10-Webhooks.md
  • components/operator/docs/05-Infrastructure services/01-PostgreSQL.md
  • components/operator/docs/05-Infrastructure services/02-Message broker.md
  • components/operator/docs/05-Infrastructure services/03-Elasticsearch.md
  • components/operator/docs/05-Infrastructure services/04-Temporal.md
  • components/operator/docs/06-Observability/01-Configure OpenTelemetry.md
  • components/operator/docs/07-Upgrade/01-Upgrade from the operator.md
  • components/operator/docs/07-Upgrade/02-Database update.md
  • components/operator/docs/08-Troubleshooting.md
  • components/operator/docs/09-Configuration reference/01-Settings.md
  • components/operator/docs/09-Configuration reference/02-Custom Resource Definitions.md
  • components/operator/hack/boilerplate.go.txt
  • components/operator/helm/crds/.helmignore
  • components/operator/helm/crds/README.md
🔇 Additional comments (7)
.github/workflows/main.yml (4)

Line range hint 1-13: LGTM: Workflow trigger and concurrency settings are well-configured.

The workflow trigger events and concurrency settings are appropriate for a CI/CD pipeline. The configuration ensures that the workflow runs on relevant events and avoids unnecessary parallel executions.


Line range hint 1-61: Please clarify the rationale behind workflow simplification and its impact on CI/CD.

The workflow has been significantly simplified by removing several jobs (Tests, Extract, GoReleaser, Deploy) that were previously part of the CI/CD process. While simplification can be beneficial, it's important to ensure that all necessary checks and deployments are still being performed.

Could you please provide more context on:

  1. The rationale behind removing these jobs?
  2. How the removed functionality (e.g., testing, releasing, deploying) is being handled now?
  3. The impact of these changes on the overall CI/CD process?

To help verify the impact, please run the following script to check for any related configuration files or scripts that might have been moved or modified:

#!/bin/bash
# Description: Search for files related to the removed jobs

echo "Searching for test-related files:"
fd -t f -e sh -e py -e js test

echo "Searching for release-related files:"
fd -t f -e sh -e yml -e yaml release

echo "Searching for deployment-related files:"
fd -t f -e sh -e yml -e yaml deploy

Line range hint 15-37: LGTM: Dirty job is well-structured. Verify custom runner setup.

The Dirty job is well-configured with appropriate steps for code quality checks. The use of pre-commit hooks and checking for uncommitted changes are good practices.

Please verify that the custom runner "formance-runner" is properly set up and available for this workflow. Run the following script to check the available runners:


Line range hint 39-61: LGTM: TestsIntegration job is well-structured. Clarify SPEAKEASY_API_KEY usage.

The TestsIntegration job is properly configured and depends on the Dirty job, ensuring code quality before running integration tests. The use of earthly for consistency is commendable.

Could you please clarify the purpose and necessity of the SPEAKEASY_API_KEY in the integration tests? If it's not required, consider removing it to enhance security. If it is required, ensure it's properly documented.

To verify the usage of SPEAKEASY_API_KEY, run the following script:

✅ Verification successful

: The SPEAKEASY_API_KEY is appropriately utilized within the Earthfile configurations for the TestsIntegration job. No further action is required.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Search for SPEAKEASY_API_KEY usage in the codebase

rg --type yaml --type dockerfile --type bash -i 'SPEAKEASY_API_KEY'

Length of output: 358


Script:

#!/bin/bash
# Description: Comprehensive search for SPEAKEASY_API_KEY usage across the entire codebase

rg -i 'SPEAKEASY_API_KEY'

Length of output: 287

Earthfile (3)

12-14: Approve new imports with a clarification request.

The addition of these new imports aligns with the PR objective of moving the project to a dedicated repo. This modular approach can improve maintainability and scalability.

However, could you please clarify why the flows import is aliased as orchestration? This might lead to confusion for other developers.


Line range hint 1-186: Overall assessment: Approved with minor clarifications needed.

The changes in this Earthfile align well with the PR objective of moving the project to a dedicated repo. The new import structure and simplified build processes should improve maintainability and clarity.

Key points:

  1. New imports for flows (as orchestration), reconciliation, and wallets have been added.
  2. The build process for OpenAPI specifications has been simplified.
  3. Some BUILD commands have been removed, potentially streamlining the process.

Please address the clarifications requested in the previous comments, particularly regarding the alias usage and ensuring comprehensive test coverage is maintained.


Line range hint 180-186: Approve simplification with a request for verification.

The removal of BUILD commands for 'components' and 'ee' in the tests, pre-commit, and tidy sections aligns with the project's restructuring. This simplification can lead to a more streamlined build process.

However, please verify that all necessary tests and checks are still being performed, possibly in the individual repositories for each component. It's crucial to maintain comprehensive test coverage despite the structural changes.

@flemzord flemzord added this pull request to the merge queue Sep 26, 2024
Merged via the queue into main with commit 41a6d3a Sep 26, 2024
4 of 5 checks passed
@flemzord flemzord deleted the refactor/clear branch September 26, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants