Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more Logging for cloud functional test workflow #7885

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

lakshmimsft
Copy link
Contributor

Description

There have been some unexpected runs for the cloud-functional-tests which fail. A recent instance was on 9/1/2024, with multiple runs, some of which were against commits that were already merged into main 2 days ago.
eg. https://github.com/radius-project/radius/actions/runs/10648348010

This PR adds additional logging to capture the values of multiple fields, allowing us to determine the correct filter conditions for triggering the workflow when event = workflow_run.

Note: This specific event may also have been related to a GitHub outage that happened the same week but nonetheless, similar failed runs have been recorded earlier.

Type of change

Fixes: Gets more data for #7782

@lakshmimsft lakshmimsft requested review from a team as code owners September 4, 2024 16:35
Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.18%. Comparing base (dca3a27) to head (2cd75c9).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7885      +/-   ##
==========================================
+ Coverage   61.12%   61.18%   +0.06%     
==========================================
  Files         523      523              
  Lines       27632    27683      +51     
==========================================
+ Hits        16889    16938      +49     
- Misses       9251     9252       +1     
- Partials     1492     1493       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@radius-functional-tests
Copy link

radius-functional-tests bot commented Sep 4, 2024

Radius functional test overview

🔍 Go to test action run

Name Value
Repository lakshmimsft/radius-mainfork
Commit ref bbfe57b
Unique ID func31a11f647a
Image tag pr-func31a11f647a
Click here to see the list of tools in the current test run
  • gotestsum 1.12.0
  • KinD: v0.20.0
  • Dapr: 1.12.0
  • Azure KeyVault CSI driver: 1.4.2
  • Azure Workload identity webhook: 1.3.0
  • Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-func31a11f647a
  • Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
  • applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-func31a11f647a
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-func31a11f647a
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func31a11f647a
  • deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting ucp-cloud functional tests...
⌛ Starting datastoresrp-cloud functional tests...
⌛ Starting corerp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ datastoresrp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded

@lakshmimsft lakshmimsft force-pushed the lakshmimsft/iderror branch 2 times, most recently from 7812968 to db01916 Compare September 5, 2024 05:45
@radius-functional-tests
Copy link

radius-functional-tests bot commented Sep 5, 2024

Radius functional test overview

🔍 Go to test action run

Name Value
Repository lakshmimsft/radius-mainfork
Commit ref 2cd75c9
Unique ID func92bcfc5d36
Image tag pr-func92bcfc5d36
Click here to see the list of tools in the current test run
  • gotestsum 1.12.0
  • KinD: v0.20.0
  • Dapr: 1.12.0
  • Azure KeyVault CSI driver: 1.4.2
  • Azure Workload identity webhook: 1.3.0
  • Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-func92bcfc5d36
  • Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
  • applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-func92bcfc5d36
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-func92bcfc5d36
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func92bcfc5d36
  • deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting corerp-cloud functional tests...
⌛ Starting datastoresrp-cloud functional tests...
⌛ Starting ucp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ datastoresrp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded

@lakshmimsft lakshmimsft merged commit dd81399 into radius-project:main Sep 5, 2024
28 checks passed
ytimocin pushed a commit that referenced this pull request Sep 6, 2024
# Description

There have been some unexpected failed runs for the
cloud-functional-tests #7782.
Earlier PR #7885 added
debug logs which have given some useful data.
This PR adds to these debug logs to help debug further.

## Type of change
- This pull request fixes a bug in Radius and has an approved issue
#7782

Fixes: Gets more data for #7782
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants