Skip to content

refactor: rename agent_config to runtime_config - #562

Open
EzgiTastan wants to merge 1 commit into
eunomia-bpf:masterfrom
EzgiTastan:refactor/rename-agent-config-to-runtime-config
Open

refactor: rename agent_config to runtime_config#562
EzgiTastan wants to merge 1 commit into
eunomia-bpf:masterfrom
EzgiTastan:refactor/rename-agent-config-to-runtime-config

Conversation

@EzgiTastan

Copy link
Copy Markdown

Addresses #330

The config struct is used for the entire runtime, not just the agent. Renamed for clarity:

  • agent_configruntime_config
  • construct_agent_config_from_env()construct_runtime_config_from_env()
  • bpftime_get/set_agent_config()bpftime_get/set_runtime_config()

33 files updated. No functional changes.

Note: syscall-server config merge can be done in a follow-up.

@yunwei37

Copy link
Copy Markdown
Member

CI triage update (2026-07-16): current visible blockers are cancelled GPU/PTX checks from an old run, and the branch is behind/conflicting. The accessible logs do not provide a current PR-local failure to patch. Next useful step is contributor rebase/conflict resolution plus fresh CI; avoid treating the cancelled GPU checks as a code diagnosis.

Addresses eunomia-bpf#330 - renames the config struct and related functions
to better reflect that it's used for the entire runtime, not just
the agent component.
Changes:
- struct agent_config -> struct runtime_config
- construct_agent_config_from_env() -> construct_runtime_config_from_env()
- bpftime_get_agent_config() -> bpftime_get_runtime_config()
- bpftime_set_agent_config() -> bpftime_set_runtime_config()
The syscall-server config merge mentioned in eunomia-bpf#330 can be addressed
in a follow-up PR.
@EzgiTastan
EzgiTastan force-pushed the refactor/rename-agent-config-to-runtime-config branch from b5fa7bb to d03214f Compare July 17, 2026 19:13
@EzgiTastan

Copy link
Copy Markdown
Author

update: rebased onto latest master and resolved the conflicts. The rename now also covers call sites added on master since I branched

@yunwei37

Copy link
Copy Markdown
Member

Fresh status after the contributor rebase: the branch is now mergeable and the broad build/test matrix is green. The only non-success checks shown on the current PR are the cancelled build-bpftime-gpu and test-ptxpass jobs; there is no PR-local failure log to patch. This is now waiting on GPU/PTX runner capacity or a maintainer-triggered fresh run, then review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants