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

CI: Telemetry for step trace is broken #32241

Open
joestringer opened this issue Apr 29, 2024 · 1 comment
Open

CI: Telemetry for step trace is broken #32241

joestringer opened this issue Apr 29, 2024 · 1 comment
Labels
area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow

Comments

@joestringer
Copy link
Member

PR #32037 introduced telemetry to several workflows.

We can see in the original PR that the "Step trace" breaks down the time contribution from various different steps throughout the job (triggered from PR):

https://github.com/cilium/cilium/actions/runs/8731358629

More recently the step trace reports that all of the steps happen at the same time for the same length (Ariane-triggered, from PR #32167):

https://github.com/cilium/cilium/actions/runs/8880222005

We should investigate and resolve the issue that is causing the step trace to incorrectly report step timestamps and duration.

@joestringer joestringer added area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow labels Apr 29, 2024
@joestringer
Copy link
Member Author

Related, this workflow run complains about misformatted gantt chart in the step trace. This is PR #32020 and a different workflow (ci-ipsec-upgrade).

https://github.com/cilium/cilium/actions/runs/8817802264

Unable to render rich display

Parse error on line 11:
...1713967134000 Call actions/checkout aga...
----------------------^
Expecting 'EOF', 'SPACE', 'NL', 'weekday_monday', 'weekday_tuesday', 'weekday_wednesday', 'weekday_thursday', 'weekday_friday', 'weekday_saturday', 'weekday_sunday', 'dateFormat', 'inclusiveEndDates', 'topAxis', 'axisFormat', 'tickInterval', 'excludes', 'includes', 'todayMarker', 'title', 'acc_title', 'acc_descr', 'acc_descr_multiline_value', 'section', 'taskTxt', 'click', got 'callbackname'

For more information, see https://docs.github.com/get-started/writing-on-github/working-with-advanced-formatting/creating-diagrams#creating-mermaid-diagrams

It's possible that the root cause of the misreported step trace data is that the plugin is failing to properly format the gantt chart for GitHub to then interpret and draw the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow
Projects
None yet
Development

No branches or pull requests

1 participant