Copilot-cli: copilot job environment

[carlcontinusys]

It looks like schedule is required when we have environments on the manifest.

We have to add
on:
schedule
on the environments too.
example:
environments:
test:
on:
schedule: "expersion"

if not added, it will generate an error schedule required.

[efekarakus]

Heya @carlcontinusys 👋

Thanks for reaching out to us!

Are you saying that when you try to write environment overrides (https://aws.github.io/copilot-cli/docs/manifest/scheduled-job/#environments) for the "test" environment you have to duplicate again the "on" section?

Is this the current behavior?

on: 
   schedule: @daily 
environments:
  test:
     on: 
        schedule: @daily 
     cpu: 512

but what you want is just:

on: 
   schedule: @daily 
environments:
  test:
     cpu: 512

Thanks!

[carlcontinusys]

Yes

[efekarakus]

Apologies for the delay on this! It completely dropped off my radar, we sent a fix for it in #2044

[camilosantana]

@efekarakus should this issue be closed since #2043 was closed with #2044 and #2043 is a duplicate of this issue?

UPDATE ignore! i was looking at a stale webUI. issue already closed. disregard!