Skip to content

Enable logging in all operator integration tests depending on ZooKeeper #317

@maltesander

Description

@maltesander

After the ZooKeeper logging spike was merged stackabletech/zookeeper-operator#588, all operator integration tests depending on ZooKeeper will fail.

This is due to CRD changes in ZooKeeper. We could simply deactivate logging, but this means T2 will not pick up any logs.

Since there are other dependencies in the integration tests (HDFS, Hive, OPA) that require changes as soon as logging is enabled in the respective products, i see two options.

Options

Disable logging for now in the integration tests using ZooKeeper and wait for all other dependencies to support logging

This would be a quick fix for failing integration tests, and we would enable logging once all other product dependencies support logging.

Adapt the dependent integration tests properly

This will offer us logs in T2 (at least for ZooKeeper) but will require changes as soon as another dependent product supports logging.

Tasks

  • Druid (ZooKeeper, HDFS, OPA)
  • HDFS (ZooKeeper)
  • HBase (ZooKeeper + HDFS)
  • Kafka (ZooKeeper + OPA)
  • NiFi (ZooKeeper)
  • Trino (ZooKeeper, HDFS, Hive, OPA)

All products with only one dependency on ZooKeeper can be properly adapted (HDFS, NiFi) already.

There are more breaking CRD changes coming coming up for ZooKeeper stackabletech/zookeeper-operator#596, so it would make sense to do this first.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions