Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Refactor integration image built#1049

Merged
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
antechrestos:refactor/build_cache_in_integration_image_build
Feb 26, 2020
Merged

Refactor integration image built#1049
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
antechrestos:refactor/build_cache_in_integration_image_build

Conversation

@antechrestos
Copy link
Copy Markdown
Contributor

Description

The aim of this change is to hide the complexity of image not being rebuilt. Image builder won't also have to know the list of dockerfiles

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes [unit tests] Not needed
  • Adds integration tests if needed.Not needed

See the contribution guide for more details.

Reviewer Notes

  • The code flow looks good.
  • Unit tests and or integration tests added.

@googlebot googlebot added the cla: yes CLA signed by all commit authors label Feb 13, 2020
Comment thread integration/images.go Outdated
@samos123 samos123 changed the title Refactor integration imaqge built Refactor integration image built Feb 15, 2020
The aim of this change is to hide the complexity of image not being rebuilt. Image builder won't also have to know the list of dockerfiles
@antechrestos
Copy link
Copy Markdown
Contributor Author

@cvgw @tejal29 I was approved . I can be merged 👼

@tejal29 tejal29 merged commit fb0cfd7 into GoogleContainerTools:master Feb 26, 2020
@antechrestos antechrestos deleted the refactor/build_cache_in_integration_image_build branch February 26, 2020 17:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes CLA signed by all commit authors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants