Skip to content

feat!: Use Tracing::pre_configured from stackable-telemetry#648

Merged
Techassi merged 6 commits intomainfrom
feat/use-tracing-pre-configured
Apr 10, 2025
Merged

feat!: Use Tracing::pre_configured from stackable-telemetry#648
Techassi merged 6 commits intomainfrom
feat/use-tracing-pre-configured

Conversation

@NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented Apr 10, 2025 •
edited
Loading

Use Tracing::pre_configured from stackable-telemetry

Part of stackabletech/issues#639.

Breaking changes:

operator-binary

  • The console log level was set by HBASE_OPERATOR_LOG, and is now set by CONSOLE_LOG.
  • The file log level was set by HBASE_OPERATOR_LOG, and is now set by FILE_LOG.
  • The file log directory was previously changed to ROLLING_LOGS, but has been updated to ROLLING_LOGS_DIR.

Testing

--- PASS: kuttl (236.33s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/logging_hbase-2.6.1_hdfs-latest-3.4.1_zookeeper-latest-3.9.3_openshift-false (236.32s)
PASS

@NickLarsenNZ
chore: Bump stackable-operator and use re-exports
2a5f0e7
@NickLarsenNZ
feat!: Use Tracing::pre_configured from stackable-telemetry
0c2d660 
BREAKING: The console log level, file log level and directory variables have been changed
@NickLarsenNZ NickLarsenNZ self-assigned this Apr 10, 2025
@NickLarsenNZ NickLarsenNZ moved this to Development: In Progress in Stackable Engineering Apr 10, 2025
@NickLarsenNZ NickLarsenNZ moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Apr 10, 2025
@NickLarsenNZ NickLarsenNZ mentioned this pull request Apr 10, 2025
Closed
Techassi
Techassi requested changes Apr 10, 2025
View reviewed changes
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Apr 10, 2025
@NickLarsenNZ NickLarsenNZ requested a review from Techassi April 10, 2025 09:18
@Techassi Techassi enabled auto-merge April 10, 2025 09:21
@Techassi Techassi added this pull request to the merge queue Apr 10, 2025
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Apr 10, 2025
Merged via the queue into main with commit 5f37aa2 Apr 10, 2025
17 checks passed
@Techassi Techassi deleted the feat/use-tracing-pre-configured branch April 10, 2025 09:37
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Techassi Techassi Techassi approved these changes

Assignees

@NickLarsenNZ NickLarsenNZ

Labels

None yet

Projects

Stackable Engineering
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NickLarsenNZ @Techassi