Skip to content

chore(cosmos): map devnet config to mainnet upgrade name for upgrading devnet to u19 #11316

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: dev-upgrade-19-devnet
Choose a base branch
from

Conversation

mujahidkay
Copy link
Member

Description

Upgrade proposal with incorrect upgrade name was passed on devnet (using agoric-upgrade-19-mainnet). This PR maps mainnet upgrade name to devnet variant config so we can use the docker image from this branch to deploy u19 on devnet.

Unsure if we actually wanna merge this in the release branch as it is just for instagoric validator nodes.

@mujahidkay mujahidkay added the force:integration Force integration tests to run on PR label Apr 23, 2025
Copy link

cloudflare-workers-and-pages bot commented Apr 23, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: e960d6e
Status: ✅  Deploy successful!
Preview URL: https://0eff7d29.agoric-sdk.pages.dev
Branch Preview URL: https://mk-devnet-intermediary.agoric-sdk.pages.dev

View logs

@mhofman
Copy link
Member

mhofman commented Apr 23, 2025

This PR should not get merged, but is ok to have a devnet specific commit living on a branch offshoot. Unfortunately we risk issues because the upgrade info referenced the release commit without this patch, we'll need to communicated this clearly.

Please rebase this branch to clean it up, then we can upgrade devnet to it.

@mhofman
Copy link
Member

mhofman commented Apr 23, 2025

Maybe we should target a dev-upgrade-19-devnet branch

@mujahidkay mujahidkay changed the base branch from dev-upgrade-19 to dev-upgrade-19-devnet April 24, 2025 09:36
@mujahidkay mujahidkay force-pushed the mk/devnet-intermediary branch 2 times, most recently from 1104a6e to 709dcdb Compare April 24, 2025 10:35
Copy link

socket-security bot commented Apr 24, 2025

No dependency changes detected. Learn more about Socket for GitHub.

👍 No dependency changes detected in pull request

@mujahidkay mujahidkay force-pushed the mk/devnet-intermediary branch from 709dcdb to dd74834 Compare April 24, 2025 10:46
@mujahidkay mujahidkay added force:integration Force integration tests to run on PR and removed force:integration Force integration tests to run on PR labels Apr 24, 2025
@mujahidkay mujahidkay marked this pull request as ready for review April 24, 2025 11:02
@mujahidkay mujahidkay requested a review from a team as a code owner April 24, 2025 11:02
@mujahidkay mujahidkay requested a review from AgoricTriage April 24, 2025 11:02
@mujahidkay mujahidkay force-pushed the mk/devnet-intermediary branch 3 times, most recently from a91fcb7 to 0fc016a Compare April 24, 2025 12:12
path: ./agoric-sdk
# Forces xsnap to initialize all memory to random data, which increases
# the chances the content of snapshots may deviate between validators
xsnap-random-init: '1'
Copy link
Member

Choose a reason for hiding this comment

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

I think this may be the problem. I don't think it should be in this cherry pick

@mujahidkay mujahidkay force-pushed the mk/devnet-intermediary branch 2 times, most recently from 25d0a57 to e960d6e Compare April 24, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants