Skip to content

Commit

Permalink
Merge pull request #2203 from hhiroshell/fix-pack-2199
Browse files Browse the repository at this point in the history
Update the Google builder in the list of known builders.
  • Loading branch information
natalieparellano committed Jul 9, 2024
2 parents 7d3a810 + 2c3d5f6 commit 0b0614b
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion acceptance/assertions/output.go
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ func (o OutputAssertionManager) IncludesTrustedBuildersHeading() {
o.assert.Contains(o.output, "Trusted Builders:")
}

const googleBuilder = "gcr.io/buildpacks/builder:v1"
const googleBuilder = "gcr.io/buildpacks/builder:google-22"

func (o OutputAssertionManager) IncludesGoogleBuilder() {
o.testObject.Helper()
Expand Down
4 changes: 2 additions & 2 deletions internal/builder/known_builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ type KnownBuilder struct {
var KnownBuilders = []KnownBuilder{
{
Vendor: "Google",
Image: "gcr.io/buildpacks/builder:v1",
DefaultDescription: "GCP Builder for all runtimes",
Image: "gcr.io/buildpacks/builder:google-22",
DefaultDescription: "Ubuntu 22.04 base image with buildpacks for .NET, Dart, Go, Java, Node.js, PHP, Python, and Ruby",
Suggested: true,
Trusted: true,
},
Expand Down
8 changes: 4 additions & 4 deletions internal/commands/config_trusted_builder_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ func testTrustedBuilderCommand(t *testing.T, when spec.G, it spec.S) {
h.AssertNil(t, command.Execute())
h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
"heroku/builder:20",
"heroku/builder:22",
"heroku/builder:24",
Expand All @@ -71,7 +71,7 @@ func testTrustedBuilderCommand(t *testing.T, when spec.G, it spec.S) {
h.AssertNil(t, command.Execute())
h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
"heroku/builder:20",
"heroku/builder:22",
"heroku/builder:24",
Expand All @@ -93,7 +93,7 @@ func testTrustedBuilderCommand(t *testing.T, when spec.G, it spec.S) {
h.AssertNotContains(t, outBuf.String(), builderName)
h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
"heroku/builder:20",
"heroku/builder:22",
"heroku/builder:24",
Expand All @@ -110,7 +110,7 @@ func testTrustedBuilderCommand(t *testing.T, when spec.G, it spec.S) {

h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
builderName,
"heroku/builder:20",
"heroku/builder:22",
Expand Down
4 changes: 2 additions & 2 deletions internal/commands/list_trusted_builders_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ func testListTrustedBuildersCommand(t *testing.T, when spec.G, it spec.S) {
h.AssertNotContains(t, outBuf.String(), builderName)
h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
"heroku/builder:20",
"heroku/builder:22",
"heroku/builder:24",
Expand All @@ -86,7 +86,7 @@ func testListTrustedBuildersCommand(t *testing.T, when spec.G, it spec.S) {

h.AssertContainsAllInOrder(t,
outBuf,
"gcr.io/buildpacks/builder:v1",
"gcr.io/buildpacks/builder:google-22",
builderName,
"heroku/builder:20",
"heroku/builder:22",
Expand Down

0 comments on commit 0b0614b

Please sign in to comment.