Skip to content

Conversation

@inteon
Copy link
Contributor

@inteon inteon commented Nov 14, 2025

These new debug log messages should give us insights in when an upload starts and how much data we are trying to upload.

This should help with debugging https://venafi.atlassian.net/browse/VC-46449.

For verbosity > 2 (-v 2), you now get these log lines:

I1112 19:29:56.802510   10415 client_file.go:32] "uploading data readings" logger="Run.gatherAndOutputData.postData" url="..." cluster_name="..." data_readings_count=0 data_size_bytes=4

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
@inteon inteon added the test-e2e To signal e2e test job to be run label Nov 14, 2025
@inteon inteon force-pushed the add_debug_log_message branch from c75d83a to 14a87d9 Compare November 14, 2025 15:29
@inteon inteon requested review from maelvls and wallrj November 14, 2025 15:40
Copy link
Member

@wallrj-cyberark wallrj-cyberark left a comment

Choose a reason for hiding this comment

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

I clicked re-run for the failed E2E test.

$ curl --fail-with-body -sS "https://${VEN_API_HOST}/v1/teams" -H "tppl-api-key: $VEN_API_KEY" | jq '.teams[0].id' -r
d2508300-3705-11ee-a17b-69a77fb429d7

And the E2E test ran successfully locally and I saw the new log message .

"cluster_name", opts.ClusterName,
"data_readings_count", len(readings),
"data_size_bytes", len(data),
)
Copy link
Member

Choose a reason for hiding this comment

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

I observed this log message by running the E2E test locally:

$ make test-e2e-gke
...
{
  "ts": 1763137282342.5798,
  "caller": "vcp_oauth/vcp_oauth.go:64",
  "msg": "acquired OAuth token from Certificate Manager SaaS API",
  "v": 0,
  "logger": "Run.gatherAndOutputData.postData",
  "scope": "",
  "token_type": "bearer",
  "valid_until": 1763137282342.5864
}
{
  "ts": 1763137282390.8809,
  "caller": "client/client_venconn.go:163",
  "msg": "uploading data readings",
  "v": 2,
  "logger": "Run.gatherAndOutputData.postData",
  "url": "https://api.venafi.cloud/v1/tlspk/upload/clusterdata/no",
  "cluster_name": "venafi-kubernetes-agent-e2e",
  "data_readings_count": 36,
  "data_size_bytes": 749416
}
{"ts":1763137282973.3423,"caller":"agent/run.go:420","msg":"Data sent successfully","v":0,"logger":"Run.gatherAndOutputData.postData"}
...

@wallrj-cyberark
Copy link
Member

Looks like the API key has expired on the VCP test tenant. I suggest merging anyway. The E2E test passed for me locally.

@inteon inteon merged commit c847309 into master Nov 14, 2025
4 of 6 checks passed
@wallrj-cyberark wallrj-cyberark deleted the add_debug_log_message branch November 14, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-e2e To signal e2e test job to be run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants