8000 fix: do multistage builds for e2e-node (backport #5220) by mergify[bot] · Pull Request #5221 · cometbft/cometbft · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: do multistage builds for e2e-node (backport #5220) #5221

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

Merged
merged 2 commits into from
Jun 23, 2025

Conversation

mergify[bot]
Copy link
Contributor
@mergify mergify bot commented Jun 18, 2025

I removed .git from the .dockerignore because it's used during building. Doing multistage builds should keep it out of the runtime build. Open to alternative ideas.

PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

This is an automatic backport of pull request #5220 done by [Mergify](https://mergify.com).

---

I removed .git from the .dockerignore because it's used during building.
Doing multistage builds should keep it out of the runtime build. Open to
alternative ideas.

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments

(cherry picked from commit e1a8604)

# Conflicts:
#	test/e2e/docker/Dockerfile
#	test/e2e/docker/Dockerfile.debug
@mergify mergify bot added the conflicts label Jun 18, 2025
@mergify mergify bot requested review from a team as code owners June 18, 2025 01:56
@mergify mergify bot added the conflicts label Jun 18, 2025
Copy link
Contributor Author
mergify bot commented Jun 18, 2025

Cherry-pick of e1a8604 has failed:

On branch mergify/bp/v1.x/pr-5220
Your branch is up to date with 'origin/v1.x'.

You are currently cherry-picking commit e1a86048.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   .dockerignore

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   test/e2e/docker/Dockerfile
	both modified:   test/e2e/docker/Dockerfile.debug

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@zrbecker zrbecker merged commit 848eade into v1.x Jun 23, 2025
1 check was pending
@zrbecker zrbecker deleted the mergify/bp/v1.x/pr-5220 branch June 23, 2025 18:42
@zrbecker zrbecker restored the mergify/bp/v1.x/pr-5220 branch June 23, 2025 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0