bottlerocket-os / bottlerocket-os/bottlerocket-test-system

Publish `merge-toml` as a crate instead of copying code

Open
#707 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
18
Forks
27
PR merge metrics
No merged PRs in 30d

Description

Currently `bottlerocket/agents/src/userdata.rs` contains a lot of code from the `merge-toml` crate found here: https://github.com/bottlerocket-os/bottlerocket/blob/v1.11.1/sources/api/storewolf/merge-toml/src/lib.rs. Instead of copying this code, the `merge-toml` crate should be published to `crates.io` so it can be used as a dependency in TestSys.

Contributor guide

Open the contributing guide

Research direction

Compare bottlerocket/agents/src/userdata.rs with the merge-toml implementation in sources/api/storewolf/merge-toml/src/lib.rs. Start by determining the crate's publishing requirements and how TestSys should consume it as a dependency. Done means the copied implementation is replaced by the published crates.io dependency and TestSys continues to use the same functionality.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
build-system, developer-experience
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.