Skip to content

Commit

Permalink
fix(agd): upgrade all orchestration vats to new liveslots
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelfig committed Dec 23, 2024
1 parent 837776e commit 7362f81
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 5 deletions.
10 changes: 5 additions & 5 deletions a3p-integration/proposals/n:upgrade-next/initial.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ import '@endo/init/debug.js';
import { getVatDetails } from '@agoric/synthetic-chain';

const vats = {
network: { incarnation: 1 },
ibc: { incarnation: 1 },
localchain: { incarnation: 1 },
orchestration: { incarnation: 0 },
transfer: { incarnation: 1 },
network: { incarnation: 2 },
ibc: { incarnation: 2 },
localchain: { incarnation: 2 },
orchestration: { incarnation: 1 },
transfer: { incarnation: 2 },
walletFactory: { incarnation: 5 },
zoe: { incarnation: 3 },
};
Expand Down
8 changes: 8 additions & 0 deletions golang/cosmos/app/upgrade.go
Original file line number Diff line number Diff line change
Expand Up @@ -233,6 +233,14 @@ func unreleasedUpgradeHandler(app *GaiaApp, targetUpgrade string) func(sdk.Conte
),
)

// Upgrade vats using Vows in Upgrade 18 in order to use a new liveslots that
// avoids a memory leak in watchPromise.
CoreProposalSteps = append(CoreProposalSteps,
vm.CoreProposalStepForModules(
"@agoric/builders/scripts/vats/upgrade-orchestration.js",
),
)

// CoreProposals for Upgrade 19. These should not be introduced
// before upgrade 18 is done because they would be run in n:upgrade-next
//
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ export const defaultProposalBuilder = async ({ publishRef, install }) =>
network: publishRef(install('@agoric/vats/src/vat-network.js')),
localchain: publishRef(install('@agoric/vats/src/vat-localchain.js')),
transfer: publishRef(install('@agoric/vats/src/vat-transfer.js')),
orchestration: publishRef(
install('@agoric/orchestration/src/vat-orchestration.js'),
),
},
},
],
Expand Down

0 comments on commit 7362f81

Please sign in to comment.