Skip to content

Conversation

@Anmol1696
Copy link
Collaborator

Part of: #695

@Anmol1696
Copy link
Collaborator Author

Something is wrong with the deleteion of so many output files, specially builder and other configmaps.

@Anmol1696 Anmol1696 requested a review from Copilot July 2, 2025 10:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds generated YAML outputs for end-to-end configuration tests across multiple chains and components.

  • Introduces full test outputs under generator/__tests__/__output__/config-tests for Cosmoshub, Hyperweb, and Evmos scenarios
  • Adds ConfigMaps for setup scripts and keys, plus Services, Deployments/StatefulSets for genesis, validator, registry, relayer, and frontend playground
  • Ensures coverage of ingress monitoring, chain initialization, and IBC relayer flows

Reviewed Changes

Copilot reviewed 131 out of 511 changed files in this pull request and generated no comments.

Show a summary per file
File(s) Description
packages/packages/generator/tests/output/config-tests/ingress-monitoring-full/cosmoshub Cosmoshub test outputs (genesis StatefulSet, Service, ConfigMaps, keys)
packages/packages/generator/tests/output/config-tests/ingress-monitoring-full/configmaps/setup-scripts.yaml Global setup-scripts ConfigMap
packages/packages/generator/tests/output/config-tests/ingress-monitoring-full/configmaps/keys.yaml Global keys ConfigMap
packages/packages/generator/tests/output/config-tests/hyperweb Hyperweb chain outputs (registry, validator, genesis, frontend playground)
packages/packages/generator/tests/output/config-tests/evmos Evmos relayer and registry test outputs (Hermes relayer, registry, ConfigMaps)
Comments suppressed due to low confidence (2)

packages/packages/generator/tests/output/config-tests/ingress-monitoring-full/cosmoshub/genesis-service.yaml:54

  • The port name "metrics" is defined twice in this Service, which will cause a validation error. Remove or rename the duplicate entry.
    - name: metrics

packages/packages/generator/tests/output/config-tests/hyperweb/hyperweb/validator-statefulset.yaml:53

  • The block scalar indicator "|2-" appears to be a typo—use "|-" or "|" for a multi-line script literal.
            - |2-

@Anmol1696 Anmol1696 merged commit ca9453b into main Jul 2, 2025
3 checks passed
@Anmol1696 Anmol1696 deleted the anmol/add-config-testing branch July 2, 2025 11:03
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