Skip to content

COMPUTE-1695 track dxfuse user agent - #108

Merged
YuxinShi0423 merged 14 commits into
masterfrom
COMPUTE-1695-track-dxfuse-user-agent
Sep 26, 2025
Merged

YuxinShi0423 merged 14 commits into
masterfrom
COMPUTE-1695-track-dxfuse-user-agent

Conversation

@YuxinShi0423

Copy link
Copy Markdown
Contributor

No description provided.

* rename types

* refactor proj find to return description

* mv project find by name to manifest

* add allowOverwrite flag

* fix wait ready check

* rename validate mountpoint func

* pass allowOverwrite to fuse mountconfig

* test log

* fix readOnly

* rename buildManifest
@YuxinShi0423
YuxinShi0423 changed the base branch from master to COMPUTE-1544-allow-overwrite September 19, 2025 00:54
@YuxinShi0423
YuxinShi0423 marked this pull request as draft September 19, 2025 00:54
@YuxinShi0423
YuxinShi0423 changed the base branch from COMPUTE-1544-allow-overwrite to master September 22, 2025 18:18
@YuxinShi0423
YuxinShi0423 changed the base branch from master to COMPUTE-1544-allow-overwrite September 22, 2025 18:19
@YuxinShi0423
YuxinShi0423 marked this pull request as ready for review September 22, 2025 19:53
@YuxinShi0423
YuxinShi0423 force-pushed the COMPUTE-1695-track-dxfuse-user-agent branch 2 times, most recently from b9935d2 to f6865ab Compare September 22, 2025 20:32
@YuxinShi0423
YuxinShi0423 changed the base branch from COMPUTE-1544-allow-overwrite to master September 22, 2025 20:32
@YuxinShi0423
YuxinShi0423 force-pushed the COMPUTE-1695-track-dxfuse-user-agent branch from f6865ab to b89657e Compare September 22, 2025 20:33
Comment thread doc/Procedures.md Outdated

## Track usage with Splunk logs

Launch information is automatically recorded in the `user-agent` field of `/system/greet` API call when creating a new dxFUSE session, with the following fields included:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

may be give an example of the user-agent string (with fake user ids) and document Mode variables.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

f7da126
also removed workspace

Comment thread doc/Procedures.md Outdated

Launch information is automatically recorded in the `user-agent` field of `/system/greet` API call when creating a new dxFUSE session:
```
dxfuse/v1.6.0 (OS=Ubuntu 22.04, Project=project-1111, LaunchedBy=user-BBB, MountProjects=[project-1111], Manifest=None, Workspace=container-2222, Mode=AllowOverwrite, JobId=job-1234, ExecutableName=test_app, BillTo=org-AAA)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
dxfuse/v1.6.0 (OS=Ubuntu 22.04, Project=project-1111, LaunchedBy=user-BBB, MountProjects=[project-1111], Manifest=None, Workspace=container-2222, Mode=AllowOverwrite, JobId=job-1234, ExecutableName=test_app, BillTo=org-AAA)
dxfuse/v1.6.0 (OS=Ubuntu 22.04, Project=project-1111, LaunchedBy=user-BBB, MountProjects=[project-1111], Manifest=None, Mode=AllowOverwrite, JobId=job-1234, ExecutableName=test_app, BillTo=org-AAA)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

remove workspace from the example line as it's supposed to be removed

@emiloslavsky emiloslavsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Procedures.md looks good

@YuxinShi0423
YuxinShi0423 merged commit 3b3e7ba into master Sep 26, 2025
4 checks passed
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.

2 participants