Skip to content

Conversation

frenzox
Copy link
Contributor

@frenzox frenzox commented Aug 18, 2025

No description provided.

@frenzox frenzox requested a review from a team as a code owner August 18, 2025 15:14
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR adds a new Solana program upgrade documentation file for the August 12, 2025 devnet upgrade. The document transforms the existing template into a concrete implementation guide for upgrading five Solana programs (Gateway, ITS, Gas Service, Governance, and Multicall) from version 0.1.0 to 0.2.0 in the devnet-amplifier environment.

The change follows the established pattern in the codebase for release documentation, similar to the CosmWasm and EVM release documents found in the releases directory. The document serves dual purposes: providing step-by-step upgrade instructions and acting as a tracking mechanism to monitor deployment completion across different programs.

Key additions include populated tracking tables with specific program hashes, detailed upgrade procedures, and a new ITS initialization step that wasn't present in the template. This initialization step suggests the upgrade includes breaking changes that require re-initialization of the ITS service, aligning with the pattern seen in other major version upgrades throughout the codebase.

Important Files Changed

File Changes
Filename Score Overview
releases/solana/Programs-upgrade-2025-08-12.md 3/5 Creates Solana devnet upgrade documentation with specific version numbers, hashes, and upgrade procedures

Confidence score: 3/5

  • This PR is moderately safe to merge but contains some inconsistencies that should be addressed
  • Score reflects documentation-only changes with no code impact, but inconsistencies between title and content lower confidence
  • Pay close attention to the discrepancy between the filename date (2025-08-12) and title date (August 11, 2025), plus the mismatch in the ITS initialization description

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI as "Solana CLI"
    participant Build as "Build Environment"
    participant Program as "Solana Program"
    participant Verifier as "solana-verify"
    participant E2E as "E2E Tests"

    User->>Build: "Set environment variables (BASE_IMAGE, ENV, CHAIN_ID)"
    User->>Build: "solana-verify build --base-image $BASE_IMAGE --library-name <library_name>"
    Build-->>User: "Program binaries built"
    
    User->>CLI: "solana config set --url <mainnet|devnet>"
    CLI-->>User: "Cluster configured"
    
    User->>CLI: "./solana/solana-axelar-cli upgrade --program <gateway|gas-service|governance|its> $PROGRAM_BYTECODE_PATH"
    CLI->>Program: "Upgrade program with new bytecode"
    Program-->>CLI: "Program upgraded"
    CLI-->>User: "Upgrade completed"
    
    User->>CLI: "solana/solana-axelar-cli send its init --operator <ITS_OPERATOR_BASE58_PUBKEY>"
    CLI->>Program: "Re-initialize ITS"
    Program-->>CLI: "ITS initialized"
    CLI-->>User: "ITS re-initialized"
    
    alt If mainnet deployment
        User->>Verifier: "solana-verify verify-from-repo --remote --program-id $PROGRAM_ID"
        Verifier->>Program: "Verify program bytecode"
        Program-->>Verifier: "Verification result"
        Verifier-->>User: "Program verified"
    end
    
    User->>Program: "Run GMP test transaction"
    Program-->>User: "Test transaction completed"
    
    User->>E2E: "Run e2e repository pipeline"
    E2E->>Program: "Execute end-to-end tests"
    Program-->>E2E: "Test results"
    E2E-->>User: "E2E tests completed"
Loading

1 file reviewed, 2 comments

Edit Code Review Bot Settings | Greptile

@@ -0,0 +1,129 @@
# Solana Program upgrade tracking doc <Date> (this is template)
Copy link
Contributor

Choose a reason for hiding this comment

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

style: Title contains placeholder text '<Date> (this is template)' but should reflect the actual date 2025-08-12

Suggested change
# Solana Program upgrade tracking doc <Date> (this is template)
# Solana Program upgrade tracking doc 2025-08-12


4. Initialize ITS:

Due to recent changes, we must re-initialise the gas service.
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: Documentation states 're-initialise the gas service' but the command initializes ITS - this is inconsistent

Suggested change
Due to recent changes, we must re-initialise the gas service.
Due to recent changes, we must re-initialise the ITS.

@frenzox frenzox changed the title docs: Solana devnet upgrade docs: solana devnet upgrade 20250811 Aug 18, 2025
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