Skip to content

Conversation

@pavolloffay
Copy link
Member

Version(s):

RHOSDT 3.8
OCP 4.12 - 4.21 (all supported OCP versions)

Issue:

https://issues.redhat.com/browse/TRACING-5677

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

Upstream docs https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/googlecloudexporter

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 29, 2025
@openshift-ci
Copy link

openshift-ci bot commented Sep 29, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@pavolloffay
Copy link
Member Author

/test all

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 29, 2025
* link:https://cloud.google.com/monitoring?hl=en[Google Cloud Monitoring]
* link:https://cloud.google.com/logging?hl=en[Google Cloud Logging]
* link:https://cloud.google.com/trace?hl=en[Google Cloud Trace]
* link:https://cloud.google.com/iam/docs/workload-identity-federation-with-kubernetes#deploy[Google Workflow Identity Federation docs]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@IshwarKanse are you using these commands to configure WIF?

gcloud iam workload-identity-pools create-cred-config \
    projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/POOL_ID/providers/WORKLOAD_PROVIDER_ID \
    --service-account=SERVICE_ACCOUNT_EMAIL \
    --credential-source-file=/var/run/service-account/token \
    --credential-source-type=text \
    --output-file=credential-configuration.json

Is it possible to use just the k8s SA to push data to the google cloud? e.g. without using the key.json?

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Sep 29, 2025

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay pavolloffay changed the title Wif aws googlecouldexporter RHOSDT 3.8, Google Cloud exporter and AWS IAM Sep 29, 2025
@pavolloffay pavolloffay changed the title RHOSDT 3.8, Google Cloud exporter and AWS IAM TRACING-5555 | RHOSDT 3.8, Google Cloud exporter and AWS IAM/ Sep 29, 2025
@pavolloffay
Copy link
Member Author

/test all

region: <aws_region_of_log_stream> # <3>
endpoint: <protocol><service_endpoint_of_amazon_cloudwatch_logs> # <4>
log_retention: <supported_value_in_days> # <5>
role_arn: "<iam_role>" # <6>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added googlecloud exporter test which used WIF openshift/distributed-tracing-qe#117 We have aws tests using role ARN as well.

@pavolloffay pavolloffay changed the title TRACING-5555 | RHOSDT 3.8, Google Cloud exporter and AWS IAM/ TRACING-5677 | RHOSDT 3.8, Google Cloud exporter and AWS IAM/ Sep 29, 2025
@openshift-ci
Copy link

openshift-ci bot commented Sep 29, 2025

@pavolloffay: 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/validate-asciidoc 72d557f link true /test validate-asciidoc
ci/prow/validate-portal 72d557f link true /test validate-portal

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 6, 2025
@openshift-merge-robot
Copy link

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@max-cx
Copy link
Contributor

max-cx commented Nov 20, 2025

@pavolloffay, you are welcome to close this Draft PR because I opened #102760.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants