Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat issue #98: merging jobs and bootstrap section #99

Merged
merged 2 commits into from
Aug 1, 2024

Conversation

idrissneumann
Copy link
Collaborator

Fix #98

@idrissneumann
Copy link
Collaborator Author

Test helm template with this value file:

---
config:
  default_index_root_uri: s3://qw-indexes
  storage:
    s3:
      endpoint: https://s3.fr-par.scw.cloud
      region: fr-par
      access_key_id: SCWXXXXXX
      secret_access_key: XXXXXXXXX

environment:
  QW_METASTORE_URI: s3://qw-indexes

# Seed configuration
seed:
  indexes:
    - version: 0.8
      index_id: my-index
      doc_mapping:
        field_mappings:
          - name: timestamp
            type: datetime
            fast: true
            input_formats:
              - unix_timestamp
            output_format: unix_timestamp_secs
          - name: body
            type: text
        timestamp_field: timestamp
      search_settings:
        default_search_fields: [body]
      indexing_settings:
        merge_policy:
          type: limit_merge
          max_merge_ops: 3
          merge_factor: 10
          max_merge_factor: 12

  sources: []

bootstrap:
  enabled: true
  sources:
    initContainers: &initContainers
      - name: hello
        image: busybox:1.28
        command: ['echo', 'heyyyyy']
  indexes:
    initContainers: *initContainers

Result:

$ helm template . --values values.yaml --values values2.yaml --namespace=quickwit|grep -i initContainers -A5
      initContainers:
        - command:
          - echo
          - heyyyyy
          image: busybox:1.28
          name: hello

@idrissneumann
Copy link
Collaborator Author

Tests on kind with the same value file:

$ helm template . --values values.yaml --values values2.yaml --namespace=quickwit|kubectl -n quickwit apply -f -
$ kubectl -n quickwit logs release-name-quickwit-index-my-index-8npw8 -c hello
heyyyyy
$ kubectl -n quickwit port-forward release-name-quickwit-searcher-0 7280:7280

=> Init container has been successfully run ✅
=> The index has been successfully created on quickwit ✅

Screenshot 2024-08-01 at 11 17 09

charts/quickwit/Chart.yaml Outdated Show resolved Hide resolved
@idrissneumann idrissneumann force-pushed the feat-merge-jobs-bootstrap-section branch from 183e7c3 to 1e82864 Compare August 1, 2024 13:45
@idrissneumann idrissneumann merged commit 44056e3 into main Aug 1, 2024
1 check passed
@idrissneumann idrissneumann deleted the feat-merge-jobs-bootstrap-section branch August 1, 2024 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge jobs top level section with bootstrap section
2 participants