Skip to content

CLDSRV-835: Add troubleshooting details to CI #6106

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.

Sign up for GitHub

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

Merged
bert-e merged 3 commits into from
Mar 18, 2026
Merged

CLDSRV-835: Add troubleshooting details to CI #6106

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
7676087
CLDSRV-835: Add mongodb logs to CI artifacts
BourgoisMickael Mar 12, 2026
e83e6f1
CLDSRV-835: Add telemetry to action summary
BourgoisMickael Mar 12, 2026
0b93adb
CLDSRV-835: Workflow telemetry use commit sha
BourgoisMickael Mar 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .github/actions/setup-ci/action.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,3 +52,12 @@ runs:
run: |
cat config.json | jq -r '.rateLimiting = {enabled: true, nodes: 6, serviceUserArn: "arn:aws:iam::123456789013:root"}' > config.json.new
mv config.json.new config.json
- name: Collect Workflow Telemetry
# uses: catchpoint/workflow-telemetry-action@v2
# Use a fork that fixes chart issue
# https://github.com/catchpoint/workflow-telemetry-action/pull/98
uses: whywaita/workflow-telemetry-action@use_quickchart
Copy link
Copy Markdown

@claude claude Bot Mar 13, 2026

Choose a reason for hiding this comment

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

Supply chain risk: This pins to a branch (@use_quickchart) on a third-party fork. The branch can be force-pushed at any time with arbitrary code that runs in your CI. Pin to a specific commit SHA instead.

— Claude Code

Copy link
Copy Markdown
Contributor

@tcarmet tcarmet Mar 17, 2026

Choose a reason for hiding this comment

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

I think that's fair, configuring it to the sha for now would be good and eventually when we know more about the state of the telemetry action we'll do something about it.

Or fork it inside the org (if you don't have permissions to do so let me know and I can do it for you).

Copy link
Copy Markdown
Contributor Author

@BourgoisMickael BourgoisMickael Mar 18, 2026

Choose a reason for hiding this comment

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

Replaced branch by commit in 0b93adb

Comment thread
BourgoisMickael marked this conversation as resolved.
Outdated
with:
theme: dark
comment_on_pr: false
proc_trace_chart_show: false
2 changes: 2 additions & 0 deletions .github/docker/docker-compose.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,8 @@ services:
network_mode: "host"
profiles: ['mongo', 'ceph']
image: ${MONGODB_IMAGE}
volumes:
- /tmp/artifacts/${JOB_NAME}:/logs
ceph:
network_mode: "host"
profiles: ['ceph']
Expand Down
4 changes: 4 additions & 0 deletions .github/docker/mongodb/conf/mongo-run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,8 @@ init_RS() {
}
init_RS &

# Pre-create log file as world-readable; mongod (logAppend: true) opens it without resetting permissions
# Otherwise mongod create the file with 600 permissions, preventing CI artifact upload
touch /logs/mongod.log && chmod 644 /logs/mongod.log

mongod --bind_ip_all --config=/conf/mongod.conf
4 changes: 4 additions & 0 deletions .github/docker/mongodb/conf/mongod.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
systemLog:
destination: file
path: "/logs/mongod.log"
logAppend: true
storage:
journal:
enabled: true
Expand Down
Loading