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

Added a KanikoStage type for each stage of a Dockerfile #320

Merged
merged 4 commits into from
Sep 7, 2018

Conversation

priyawadhwa
Copy link
Collaborator

@priyawadhwa priyawadhwa commented Aug 27, 2018

I added a KanikoStage to hold each stage of the Dockerfile along with
information about each stage that would be useful later on.

The new KanikoStage type holds the stage itself, along with some
additional information:

  1. FinalStage -- whether the current stage is the final stage
  2. BaseImageStoredLocally/BaseImageIndex -- whether the base image for
    this stage is stored locally, and if so what the index of the base image
    is
  3. SaveStage -- whether this stage needs to be saved for use in a future
    stage

This is the first part of a larger refactor for building stages, which
will later make it easier to add layer caching. (#300)

Priya Wadhwa added 2 commits August 27, 2018 14:15
I added a KanikoStage to hold each stage of the Dockerfile along with
information about each stage that would be useful later on.

The new KanikoStage type holds the stage itself, along with some
additional information:

1. FinalStage -- whether the current stage is the final stage
2. BaseImageStoredLocally/BaseImageIndex -- whether the base image for
this stage is stored locally, and if so what the index of the base image
is
3. SaveStage -- whether this stage needs to be saved for use in a future
stage

This is the first part of a larger refactor for building stages, which
will later make it easier to add layer caching.
@dlorenc
Copy link
Collaborator

dlorenc commented Aug 30, 2018

Looks like this needs a rebase

@priyawadhwa
Copy link
Collaborator Author

Done

@priyawadhwa priyawadhwa merged commit 4dc3434 into GoogleContainerTools:master Sep 7, 2018
@priyawadhwa priyawadhwa deleted the stages branch September 7, 2018 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants