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

Found a bug #1679

Closed
Tracked by #1646
p0mvn opened this issue Jun 6, 2022 · 4 comments
Closed
Tracked by #1646

Found a bug #1679

p0mvn opened this issue Jun 6, 2022 · 4 comments
Assignees
@niccoloraspa
Labels
ClickUp Test T:build T:dev-UX T:story A story belongs to an epic

Comments

@p0mvn
Copy link
Member

p0mvn commented Jun 6, 2022 •
edited by mrank-clickup
Loading

Im having a problem with this.
@p0mvn p0mvn mentioned this issue Jun 6, 2022
Closed
5 tasks
@daniel-farina daniel-farina moved this to Needs Review 🔍 in Osmosis Chain Development Jun 6, 2022
@p0mvn p0mvn added T:story A story belongs to an epic T:dev-UX T:build labels Jun 6, 2022
This was referenced Jun 6, 2022
Closed
Closed
Closed
Closed
This was referenced Aug 17, 2022
Merged
Merged
@niccoloraspa
Copy link
Member

niccoloraspa commented Aug 27, 2022 •
edited
Loading

At the time of the issue we had the following Dockerimages:

❯ find . -iname "*dockerfile*"

./Dockerfile
./tests/localosmosis/Dockerfile
./tests/localosmosis/mainnet_state/Dockerfile-stateExport
./tests/e2e/initialization/init.Dockerfile
./contrib/images/osmobuilder/Dockerfile
./contrib/images/rbuilder/Dockerfile

Fixing #2482 removed the contrib folder and the following images:

./contrib/images/osmobuilder/Dockerfile
./contrib/images/rbuilder/Dockerfile

@niccoloraspa niccoloraspa self-assigned this Aug 27, 2022
@niccoloraspa niccoloraspa mentioned this issue Aug 27, 2022
Merged
@niccoloraspa
Copy link
Member

Removed also:

./tests/localosmosis/Dockerfile
./tests/localosmosis/mainnet_state/Dockerfile-stateExport

@ValarDragon
Copy link
Member

Seeing that theres only two dockerfiles remaining, can we close this issue?

@niccoloraspa
Copy link
Member

Yes. Closing

Repository owner moved this from Needs Review 🔍 to Done ✅ in Osmosis Chain Development Sep 26, 2022
@mrank-clickup mrank-clickup changed the title [Story][Dockerfile Improvements] Generalize and reduce the number of Dockerfiles Found a bug Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niccoloraspa niccoloraspa

Labels
ClickUp Test T:build T:dev-UX T:story A story belongs to an epic
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@niccoloraspa @ValarDragon @p0mvn @mrank-clickup