Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Add target as an attribute to dockerfile_build #1944

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

jakegut
Copy link
Contributor

@jakegut jakegut commented Oct 29, 2021

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

I wasn't able to build a specific stage of a Dockerfile with the current implementation of dockerfile_build

Issue Number: N/A

What is the new behavior?

You can optionally specify a target attribute for dockerfile_build that will tell docker to build to that stage. Docker documentation here

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@gravypod
Copy link
Collaborator

gravypod commented Nov 9, 2021

@jakegut, mind rebasing this PR? CI should be green at head. Once it's rebased I can review (I think this looks pretty good to me).

@jakegut jakegut force-pushed the jake/dockerfile-build-target branch from 2089826 to 05b10e7 Compare November 9, 2021 03:11
@jakegut
Copy link
Contributor Author

jakegut commented Nov 9, 2021

@gravypod Done.

@alexeagle alexeagle merged commit 61e2df8 into bazelbuild:master Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants