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.

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

bundle cost is 100x too high in latest image #121

Closed
dckc opened this issue Mar 7, 2024 · 0 comments · Fixed by #128
Closed

bundle cost is 100x too high in latest image #121

dckc opened this issue Mar 7, 2024 · 0 comments · Fixed by #128
Assignees

Comments

@dckc
Copy link
Member

dckc commented Mar 7, 2024

Steps to reproduce

start a container with...

docker run --rm -ti -p 26657:26657 -p 26656:26656 -p 1317:1317 ghcr.io/agoric/agoric-3-proposals:latest

and then run the test for bundle storage cost:

~/projects/agoric-3-proposals/proposals/61:lo-bundle-cost$ ./test.sh 
Expected cost 20000000, got 2000000000

Expected results

as with the use-lo-bundle-cost image...

docker run --rm -ti -p 26657:26657 -p 26656:26656 -p 1317:1317 ghcr.io/agoric/agoric-3-proposals:use-lo-bundle-cost
~/projects/agoric-3-proposals/proposals/61:lo-bundle-cost$ ./test.sh 
Cost is 20000000

Versions etc

2024-03-06 15:55 989f22d Merge pull request #75 from Agoric/dc-bundle-cost

$ docker image ls --digests
REPOSITORY                          TAG                  DIGEST                                                                    IMAGE ID       CREATED        SIZE
ghcr.io/agoric/agoric-3-proposals   use-lo-bundle-cost   sha256:4a89ddeb74ce89d4a2df5ecfaefae4e9903bc5efb0ce67f44a2d433508bf97bd   76c61505ffe2   20 hours ago   5.89GB
ghcr.io/agoric/agoric-3-proposals   latest               sha256:a1ab29d5dc0c4047607583278e972a9d5a411607f74b369535717a4d62d23e2e   22ed0e13ae82   2 weeks ago    3.3GB
@turadg turadg self-assigned this Mar 7, 2024
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 a pull request may close this issue.

2 participants