Skip to content

WIP#11

Merged
nayrosk merged 4 commits into
mainfrom
develop
Apr 9, 2026
Merged

WIP#11
nayrosk merged 4 commits into
mainfrom
develop

Conversation

@nayrosk
Copy link
Copy Markdown
Member

@nayrosk nayrosk commented Apr 9, 2026

Description

A clear and concise description of the PR.
Use this section for review hints, explanations or discussion points/todos.

  • Summary of changes
  • Reasoning
  • Additional context

Closes {LINK TO GH ISSUE}

Type of change

Check options that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Security (fix a security-related issue)
  • Refactor (modifying code that does not involve changing functionality or fixing bugs)

Breaking change details (if applicable)

Describe the breaking change and the migration path for existing users.

Checks to complete

  • Branch is up-to-date with origin
  • I have performed a self-review of my code
  • PR title is in a conventional commit style
  • My code follows the styleguide of the project
  • All my commits are GPG signed
  • CHANGELOG.md has been updated
  • New dependencies are license-compatible

Rust checks

  • All tests pass (cargo test)
  • No compiler warnings (cargo build)
  • Deny passes (cargo deny check all)
  • Audit passes (cargo audit)
  • Mutation tests pass (cargo mutants --check)
  • Clippy passes (cargo clippy -- -D warnings)
  • Code is formatted (cargo fmt --check)
  • All public items have doc comments
  • No unsafe blocks
  • No unwrap() (except in tests)
  • No commented-out code or debug statements
  • No hardcoded credentials

MUST build on all MANDATORY toolchains:

  • x86_64-unknown-linux-gnu
  • x86_64-unknown-linux-musl
  • aarch64-unknown-linux-gnu
  • aarch64-unknown-linux-musl
  • aarch64-apple-darwin

Documentation

  • I have updated relevant documentation
  • New CLI flags or config options are documented

Summary by CodeRabbit

Release Notes

  • New Features
    • Added Cosmos Hub Gaia node container image recipe supporting multiple architectures, database backends, and build configurations
    • Added Kyve node container image recipe with network profiles and customizable build options

@nayrosk nayrosk self-assigned this Apr 9, 2026
@nayrosk nayrosk merged commit 915ce23 into main Apr 9, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: a912a8e9-f542-420e-a713-0bce672298e3

📥 Commits

Reviewing files that changed from the base of the PR and between baee34d and fe7bb4f.

⛔ Files ignored due to path filters (1)
  • assets/Logo.svg is excluded by !**/*.svg
📒 Files selected for processing (2)
  • recipes/cosmos-gaiad.toml
  • recipes/kyve-kyved.toml

📝 Walkthrough

Walkthrough

This pull request introduces two new Dockermint recipe configuration files: cosmos-gaiad.toml for the Cosmos Hub Gaia node and kyve-kyved.toml for the Kyve kyved binary. Both recipes define Docker image build specifications, including supported architectures, database backends, binary types, build variables, linker configurations, and OCI image labels.

Changes

Cohort / File(s) Summary
Cosmos Gaia Recipe
recipes/cosmos-gaiad.toml
New Dockermint recipe defining build configuration for Cosmos Hub Gaia/Gaiad node Docker image. Includes metadata, build flavours (architecture, database backend, linkage type, environment), build-time variables extracted from Go modules and Git, builder installation steps, conditional static library downloads, linker flags for dynamic/static builds, non-root user configuration, copy rules for binary and wasmvm library, port exposure, OCI image labels, and tag formatting.
Kyve Recipe
recipes/kyve-kyved.toml
New Dockermint recipe for Kyve kyved binary Docker image. Defines metadata, build flavours including network selection (mainnet/kaon), network profiles with chain constants, build variables for component versions, builder installation, static/dynamic linker configurations, user setup, binary copy rules, port exposure, OCI labels, and image tag format.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Chore/add assets and recipes #10: Directly related PR that adds the same recipes/cosmos-gaiad.toml and recipes/kyve-kyved.toml files with identical Dockermint recipe configurations.

Poem

🐰 Two recipes hop into the fold,
Cosmos and Kyve, stories to be told,
With flavours and flags and Docker delight,
These configs build images so wonderfully right!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant