Skip to content

Commit

Permalink
fix(cli): don't log platform msg unless both redirected and non-RDWS (a…
Browse files Browse the repository at this point in the history
…ws#2770)

Previously, any non-App Runner service threw out the log statement about the platform field in the manifest. 
Now, this happens only when the platform has been redirected _and_ a non-App Runner service has been created.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
  • Loading branch information
huanjani authored and thrau committed Dec 9, 2022
1 parent 1eebab9 commit 0e3030a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions internal/pkg/cli/svc_init.go
Original file line number Diff line number Diff line change
Expand Up @@ -253,9 +253,9 @@ func (o *initSvcOpts) Execute() error {
o.platform = platform
if o.platform != nil {
log.Warningf(`Your architecture type is currently unsupported. Setting platform %s instead.\n`, dockerengine.DockerBuildPlatform(dockerengine.LinuxOS, dockerengine.Amd64Arch))
}
if o.wkldType != manifest.RequestDrivenWebServiceType {
log.Warning("See 'platform' field in your manifest.\n")
if o.wkldType != manifest.RequestDrivenWebServiceType {
log.Warning("See 'platform' field in your manifest.\n")
}
}

manifestPath, err := o.init.Service(&initialize.ServiceProps{
Expand Down

0 comments on commit 0e3030a

Please sign in to comment.