omarsmak commented on code in PR #389:
URL: https://github.com/apache/polaris/pull/389#discussion_r1989656706


##########
helm/polaris/values.yaml:
##########
@@ -359,209 +359,76 @@ extraEnv:
 #        name: aws-secret
 #        key: secret_access_key
 
-# -- Extra volumes to add to the polaris pod. See 
https://kubernetes.io/docs/concepts/storage/volumes/.
-extraVolumes: []
-  # - name: extra-volume
-  #   emptyDir: {}
-
-# -- Extra volume mounts to add to the polaris container. See 
https://kubernetes.io/docs/concepts/storage/volumes/.
-extraVolumeMounts: []
-  # - name: extra-volume
-  #   mountPath: /usr/share/extra-volume
-
-# -- Add additional init containers to the polaris pod(s) See 
https://kubernetes.io/docs/concepts/workloads/pods/init-containers/.
-extraInitContainers: []
-  # - name: your-image-name
-  #   image: your-image
-  #   imagePullPolicy: Always
-  #   command: ['sh', '-c', 'echo "hello world"']
-
-tracing:
-  # -- Specifies whether tracing for the polaris server should be enabled.
-  enabled: false
-  # -- The collector endpoint URL to connect to (required).
-  # The endpoint URL must have either the http:// or the https:// scheme.
-  # The collector must talk the OpenTelemetry protocol (OTLP) and the port 
must be its gRPC port (by default 4317).
-  # See https://quarkus.io/guides/opentelemetry for more information.
-  endpoint: "http://otlp-collector:4317";
-  # -- Which requests should be sampled. Valid values are: "all", "none", or a 
ratio between 0.0 and
-  # "1.0d" (inclusive). E.g. "0.5d" means that 50% of the requests will be 
sampled.
-  # Note: avoid entering numbers here, always prefer a string representation 
of the ratio.
-  sample: "1.0d"
-  # -- Resource attributes to identify the polaris service among other tracing 
sources.
-  # See 
https://opentelemetry.io/docs/reference/specification/resource/semantic_conventions/#service.
-  # If left empty, traces will be attached to a service named "Apache 
Polaris"; to change this,
-  # provide a service.name attribute here.
-  attributes:
-    {}
-    # service.name: my-polaris
-
-metrics:
-  # -- Specifies whether metrics for the polaris server should be enabled.
-  enabled: true
-  # -- Additional tags (dimensional labels) to add to the metrics.
-  tags:
-    {}
-    # service: polaris
-    # environment: production
-
-serviceMonitor:
-  # -- Specifies whether a ServiceMonitor for Prometheus operator should be 
created.
-  enabled: true
-  # -- The scrape interval; leave empty to let Prometheus decide. Must be a 
valid duration, e.g. 1d, 1h30m, 5m, 10s.
-  interval: ""
-  # -- Labels for the created ServiceMonitor so that Prometheus operator can 
properly pick it up.
-  labels:
-    {}
-  # release: prometheus
-  # -- Relabeling rules to apply to metrics. Ref 
https://prometheus.io/docs/prometheus/latest/configuration/configuration/#relabel_config.
-  metricRelabelings: []
-    # - source_labels: [ __meta_kubernetes_namespace ]
-    #   separator: ;
-    #   regex: (.*)
-    #   target_label: namespace
-    #   replacement: $1
-    #   action: replace
-
-# -- Logging configuration.
-logging:
-  # -- The log level of the root category, which is used as the default log 
level for all categories.
-  level: INFO
-  # -- The header name to use for the request ID.
-  requestIdHeaderName: Polaris-Request-Id
-  # -- Configuration for the console appender.
-  console:
-    # -- Whether to enable the console appender.
-    enabled: true
-    # -- The log level of the console appender.
-    threshold: ALL
-    # -- Whether to log in JSON format.
-    json: false
-    # -- The log format to use. Ignored if JSON format is enabled. See
-    # https://quarkus.io/guides/logging#logging-format for details.
-    format: "%d{yyyy-MM-dd HH:mm:ss,SSS} %-5p [%c{3.}] 
[%X{requestId},%X{realmId}] [%X{traceId},%X{parentId},%X{spanId},%X{sampled}] 
(%t) %s%e%n"
-  # -- Configuration for the file appender.
-  file:
-    # -- Whether to enable the file appender.
-    enabled: false
-    # -- The log level of the file appender.
-    threshold: ALL
-    # -- Whether to log in JSON format.
-    json: false
-    # -- The log format to use. Ignored if JSON format is enabled. See
-    # https://quarkus.io/guides/logging#logging-format for details.
-    format: "%d{yyyy-MM-dd HH:mm:ss,SSS} %-5p [%c{3.}] 
[%X{requestId},%X{realmId}] [%X{traceId},%X{parentId},%X{spanId},%X{sampled}] 
(%t) %s%e%n"
-    # -- The local directory where log files are stored. The persistent volume 
claim will be mounted
-    # here.
-    logsDir: /deployments/logs
-    # -- The log file name.
-    fileName: polaris.log
-    # -- Log rotation configuration.
-    rotation:
-      # -- The maximum size of the log file before it is rotated. Should be 
expressed as a Kubernetes quantity.
-      maxFileSize: 100Mi
-      # -- The maximum number of backup files to keep.
-      maxBackupIndex: 5
-      # -- An optional suffix to append to the rotated log files. If present, 
the rotated log files
-      # will be grouped in time buckets, and each bucket will contain at most 
maxBackupIndex files.
-      # The suffix must be in a date-time format that is understood by 
DateTimeFormatter. If the
-      # suffix ends with .gz or .zip, the rotated files will also be 
compressed using the
-      # corresponding algorithm.
-      fileSuffix: ~  # .yyyy-MM-dd.gz
-    # -- The log storage configuration. A persistent volume claim will be 
created using these
-    # settings.
-    storage:
-      # -- The storage class name of the persistent volume claim to create.
-      className: standard
-      # -- The size of the persistent volume claim to create.
-      size: 512Gi
-      # -- Labels to add to the persistent volume claim spec selector; a 
persistent volume with
-      # matching labels must exist. Leave empty if using dynamic provisioning.
-      selectorLabels: {}
-        # app.kubernetes.io/name: polaris
-      # app.kubernetes.io/instance: RELEASE-NAME
-  # -- Configuration for specific log categories.
-  categories:
-    org.apache.polaris: INFO
-    org.apache.iceberg.rest: INFO
-    # Useful to debug configuration issues:
-    # io.smallrye.config: DEBUG
-  # -- Configuration for MDC (Mapped Diagnostic Context). Values specified 
here will be added to the
-  # log context of all incoming requests and can be used in log patterns.
-  mdc:
-    # aid=polaris
-    # sid=polaris-service
-    {}
-
-# -- Realm context resolver configuration.
-realmContext:
-  # -- The type of realm context resolver to use. Two built-in types are 
supported: default and test;
-  # test is not recommended for production as it does not perform any realm 
validation.
-  type: default
-  # -- List of valid realms, for use with the default realm context resolver. 
The first realm in
-  # the list is the default realm. Realms not in this list will be rejected.
-  realms:
-    - POLARIS
-
-# -- Polaris features configuration.
-features:
-  # -- Features to enable or disable globally. If a feature is not present in 
the map, the default
-  # built-in value is used.
-  defaults: {}
-    # ENFORCE_PRINCIPAL_CREDENTIAL_ROTATION_REQUIRED_CHECKING: false
-    # SUPPORTED_CATALOG_STORAGE_TYPES:
-    #   - S3
-    #   - GCS
-    #   - AZURE
-    #   - FILE
-  # -- Features to enable or disable per realm. This field is a map of maps. 
The realm name is the key, and the value is a map of
-  # feature names to values. If a feature is not present in the map, the 
default value from the 'defaults' field is used.
-  realmOverrides: {}
-    # my-realm:
-    #   ENFORCE_PRINCIPAL_CREDENTIAL_ROTATION_REQUIRED_CHECKING: true
-
-# -- Polaris persistence configuration.
-persistence:
-  # -- The type of persistence to use. Two built-in types are supported: 
in-memory and eclipse-link.
-  type: eclipse-link  # in-memory
-  # -- The configuration for the eclipse-link persistence manager.
-  eclipseLink:
-    # -- The secret name to pull persistence.xml from.
-    secret:
-      # -- The name of the secret to pull persistence.xml from.
-      # If not provided, the default built-in persistence.xml will be used. 
This is probably not what you want.
-      name: ~
-      # -- The key in the secret to pull persistence.xml from.
-      key: persistence.xml
-    # -- The persistence unit name to use.
-    persistenceUnit: polaris
-
-# -- Polaris FileIO configuration.
-fileIo:
-    # -- The type of file IO to use. Two built-in types are supported: default 
and wasb. The wasb one translates WASB paths to ABFS ones.
-  type: default
-
-# -- Storage credentials for the server. If the following properties are 
unset, default
-# credentials will be used, in which case the pod must have the necessary 
permissions to access the storage.
-storage:
-  # -- The secret to pull storage credentials from.
-  secret:
-    # -- The name of the secret to pull storage credentials from.
-    name: ~
-    # -- The key in the secret to pull the AWS access key ID from. Only 
required when using AWS.
-    awsAccessKeyId: ~
-    # -- The key in the secret to pull the AWS secret access key from. Only 
required when using AWS.
-    awsSecretAccessKey: ~
-    # -- The key in the secret to pull the GCP token from. Only required when 
using GCP.
-    gcpToken: ~
-    # -- The key in the secret to pull the GCP token expiration time from. 
Only required when using GCP. Must be a valid ISO 8601 duration. The default is 
PT1H (1 hour).
-    gcpTokenLifespan: ~
-
-# -- Polaris authentication configuration.
-authentication:
-  # -- The type of authentication to use. Two built-in types are supported: 
default and test;
-  # test is not recommended for production.
-  authenticator:
+# -- Configures whether to enable the bootstrap metastore manager job
+bootstrapMetastoreManager: false

Review Comment:
   @lefebsy maybe would be easier if you take the helm stuff outside of this PR 
and have it as a follow up. Wdyt? (so we don't prolong this PR further :))



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to