openshift / origin-aggregated-logging

139 stars 230 forks source link

LOG-2220 Fluentd collector not getting labels from /var/log/pods paths #2244

Closed alanconway closed 2 years ago

alanconway commented 2 years ago

Description

Updated the regexp used to extract labels from log paths to work with /var/log/pods or /var/log/containers/ paths.

/cc @vimalk78 /assign @jcantrill

Links

openshift-ci[bot] commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alanconway

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/origin-aggregated-logging/blob/master/OWNERS)~~ [alanconway] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
jcantrill commented 2 years ago

Please reopen against viaq/logging-fluentd

/hold

openshift-ci[bot] commented 2 years ago

@alanconway: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/smoke-5-2 abd4f0f67a58f35a83f3c7809a8c64bfca66d4e3 link true /test smoke-5-2
ci/prow/cluster-logging-operator-e2e-5-4 abd4f0f67a58f35a83f3c7809a8c64bfca66d4e3 link true /test cluster-logging-operator-e2e-5-4
ci/prow/elastic-operator-e2e-5-5 abd4f0f67a58f35a83f3c7809a8c64bfca66d4e3 link true /test elastic-operator-e2e-5-5
ci/prow/cluster-logging-operator-e2e-5-2 abd4f0f67a58f35a83f3c7809a8c64bfca66d4e3 link true /test cluster-logging-operator-e2e-5-2

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
openshift-ci[bot] commented 2 years ago

@alanconway: PR needs rebase.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
jcantrill commented 2 years ago

please reopen this PR against https://github.com/ViaQ/logging-fluentd

alanconway commented 2 years ago

Moved to https://github.com/ViaQ/logging-fluentd/pull/16