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

Schedule is invalid #394

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Schedule is invalid #394

merged 1 commit into from
Apr 9, 2024

Conversation

kingdonb
Copy link
Member

https://github.com/fluxcd/community/actions/runs/8467767683

Invalid workflow file
`schedule` accepts a list of one or more maps with the `cron` key set

(This should fix it)

@swade1987
Copy link
Member

@kingdonb do we need this at all anymore or is the plan to update for the next kubecon?

@kingdonb
Copy link
Member Author

Yes, it worked great, I want to keep the workflow around until next Kubecon, or until we can find a better home for it. If we decide to manage kubecon.md manually it would maybe work just as well, but I think it's more inclusive for community maintainers to have the Google Sites wysiwyg interface, even if it's a bit clunky.

@kingdonb
Copy link
Member Author

kingdonb commented Mar 28, 2024

Re: future plans, cross reference notes in:

@kingdonb
Copy link
Member Author

@stefanprodan 🍹

Hope you guys are having a nice trip!

https://github.com/fluxcd/community/actions/runs/8467767683
Invalid workflow file
`schedule` accepts a list of one or more maps with the `cron` key set

(This should fix it)

Signed-off-by: Kingdon P Barrett <[email protected]>
@kingdonb kingdonb force-pushed the fix-update-kubecon branch from 71407bc to cdfafc4 Compare April 9, 2024 15:10
@kingdonb kingdonb merged commit a79aa31 into main Apr 9, 2024
2 checks passed
@kingdonb kingdonb deleted the fix-update-kubecon branch April 9, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants