You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/05/12 01:23:16 UTC

[PR] generate-cve-json: update default config path to .apache-steward-overrides/ [airflow-steward]

potiuk opened a new pull request, #132:
URL: https://github.com/apache/airflow-steward/pull/132

   ## Summary
   
   [PR #392](https://github.com/apache/airflow-steward/pull/392) moved the project's `cve-json-config.toml` from `.apache-steward/tools/vulnogram/` to `.apache-steward-overrides/tools/vulnogram/` (the snapshot directory is read-only per the framework's golden rules; per-project config lives in the overrides directory). The generator's `_DEFAULT_CONFIG_RELPATH` constant in `generate-cve-json/src/generate_cve_json/cve_json.py` still pointed at the old snapshot path though, so adopters running `generate-cve-json <N> --attach` from the tracker repo without a `CVE_JSON_CONFIG` env var or `--config` flag hit:
   
   ```
   error: generate-cve-json: config file not found at <cwd>/.apache-steward/tools/vulnogram/cve-json-config.toml.
   ```
   
   This PR updates the constant + the docstring comments to match the post-#392 layout. The `--config` CLI flag path and `CVE_JSON_CONFIG` env var path are unchanged. Also updates the `conftest.py` docstring that referenced the old path.
   
   ## Files changed
   
   - `tools/vulnogram/generate-cve-json/src/generate_cve_json/cve_json.py` — `_DEFAULT_CONFIG_RELPATH` now points at `.apache-steward-overrides/...`; updated docstring + comment.
   - `tools/vulnogram/generate-cve-json/tests/conftest.py` — updated path reference in the module docstring.
   
   ## Test plan
   
   - [ ] `uv run --project tools/vulnogram/generate-cve-json pytest tools/vulnogram/generate-cve-json/tests` — 145 / 145 pass
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] generate-cve-json: update default config path to .apache-steward-overrides/ [airflow-steward]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #132:
URL: https://github.com/apache/airflow-steward/pull/132


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org