Skip to content
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.

Sign up for GitHub

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

Jump to bottom

u15: base a3p-integration on upgrade-15 #9260

Closed
ivanlei opened this issue Apr 19, 2024 · 0 comments · Fixed by #9436
Closed

u15: base a3p-integration on upgrade-15 #9260

ivanlei opened this issue Apr 19, 2024 · 0 comments · Fixed by #9436
Assignees
@gibson042
Labels
enhancement New feature or request

Comments

@ivanlei
Copy link
Contributor

ivanlei commented Apr 19, 2024

see #9206 for a similar ticket

What is the Problem Being Solved?

Description of the Design

Security Considerations

Scaling Considerations

Test Plan

Upgrade Considerations
@ivanlei ivanlei added the enhancement New feature or request label Apr 19, 2024
gibson042 added a commit that referenced this issue May 31, 2024
@gibson042
chore: Integrate on top of upgrade-15
3c12173 
Fix #9260
gibson042 added a commit that referenced this issue May 31, 2024
@gibson042
chore: Integrate on top of upgrade-15
99ae05c 
Fixes #9260
@gibson042 gibson042 mentioned this issue May 31, 2024
Merged
turadg pushed a commit that referenced this issue Jun 3, 2024
@gibson042 @turadg
chore: Integrate on top of upgrade-15
fc3df89 
Fixes #9260
@mergify mergify bot closed this as completed in #9436 Jun 3, 2024
mergify bot added a commit that referenced this issue Jun 3, 2024
@mergify
chore: Integrate on top of upgrade-15 (#9436)
440d7a8 
Fixes #9260

## Description

As of Agoric/agoric-3-proposals#157 ,
`a3p:latest` now includes results of
[upgrade-15](https://github.com/Agoric/agoric-sdk/releases/tag/agoric-upgrade-15).
This removes the duplication from the UNRELEASED upgrade handler and
a3p-integration.

### Security Considerations

n/a

### Scaling Considerations

n/a

### Documentation Considerations

n/a

### Testing Considerations

How much of the [upgrade-15
a3p-integration](https://github.com/Agoric/agoric-sdk/tree/agoric-upgrade-15/a3p-integration/proposals/a%3Aupgrade-15)
should remain in
[a:upgrade-next](https://github.com/Agoric/agoric-sdk/tree/master/a3p-integration/proposals/a%3Aupgrade-next)?
I'm thinking specifically about
[exit-reclaim.test.js](https://github.com/Agoric/agoric-sdk/blob/master/a3p-integration/proposals/a%3Aupgrade-next/exit-reclaim.test.js)
and associated preparation.

### Upgrade Considerations

@Chris-Hibbert Does
[initial.test.js](https://github.com/Agoric/agoric-sdk/blob/master/a3p-integration/proposals/a%3Aupgrade-next/initial.test.js)
needs some extension for the vaultFactory and/or scaledPriceAuthority
upgrades?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gibson042 gibson042

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Integrate on top of upgrade-15 Agoric/agoric-sdk
2 participants
@ivanlei @gibson042