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.

Sign up for GitHub

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

Jump to bottom

Beta jobs #610

Merged

johakoch merged 29 commits into master from beta-job

Dec 1, 2022

Contributor

alex-schneider commented Oct 21, 2022 •

edited by malud

Loading

Add beta_job block(s) to definitions.

Reviewer checklist

Read PR description: a summary about the changes is required
Changelog updated
Documentation: docs/{Reference, Cli, ...}, Docker and cli help/usage
Pulled branch, manually tested
Verified requirements are met
Reviewed the code
Reviewed the related tests

malud force-pushed the beta-job branch from 3cd831d to 1a856ec Compare

November 10, 2022 10:08

johakoch reviewed

View reviewed changes

docs/website/content/2.configuration/4.block/beta_job.md Outdated Show resolved Hide resolved

malud force-pushed the beta-job branch from 1a856ec to 50cb96c Compare

November 28, 2022 15:06

johakoch reviewed

View reviewed changes

docs/website/content/2.configuration/4.block/definitions.md Outdated Show resolved Hide resolved

johakoch reviewed

View reviewed changes

docs/website/content/2.configuration/4.block/job.md Outdated Show resolved Hide resolved

johakoch reviewed

View reviewed changes

config/job.go Outdated Show resolved Hide resolved

johakoch reviewed

View reviewed changes

handler/endpoint.go Outdated Show resolved Hide resolved

johakoch reviewed

View reviewed changes

config/job.go Outdated Show resolved Hide resolved

malud force-pushed the beta-job branch from a2fa828 to 8c2ab2b Compare

December 1, 2022 08:37

Alex Schneider and others added 21 commits

December 1, 2022 10:30


          Implement config

db96b5a


          Jobs have no client responses

5fa4552


          Execute jobs

4eded71


          Docu

681572d


          Set UUID into the logs

89846ee


          fixup namings

443e17d


          refactor; move jobs logic out of run cmd

6e8f70d


          Update docs/website/content/2.configuration/4.block/beta_job.md

36b7981

Co-authored-by: Johannes Koch <53434855+johakoch@users.noreply.github.com>


          refactor pkg and types

5ea8d24


          handle first run; synced vars; timer stop

e1dd8c7


          add job logs and uid field

9866b13


          add custom log for job blocks

cd06efd


          register job config struct for docs generation

05657d2


          fixup shadowed conf, default to json err tpl

186166a


          use handler iface

c83b8f0


          fixup rename

3a17850


          add log for cancel events

79f6b89


          add unit tests

a52b1db


          increase initial tick delay

5ac5e25


          add integration test; variables, custom logs

65f92d4

fixup timings

Apply suggestions from code review

Co-authored-by: Johannes Koch <53434855+johakoch@users.noreply.github.com>

fixup test expectation


          remove dummy client req from variables

7b189c7


          rm exp status from docs

45519ec

malud force-pushed the beta-job branch from 8c2ab2b to 45519ec Compare

December 1, 2022 09:30

Marcel Ludwig and others added 4 commits

December 1, 2022 10:32


          finally after master rebase; fixed method return val

d54a53d


          rm obsolete iface implementation

f12a740


          rm obsolete ep condition

fb0e2ee


          remove default value for interval

d03bb93

johakoch force-pushed the beta-job branch from be16a1e to d03bb93 Compare

December 1, 2022 11:19

Marcel Ludwig added 3 commits

December 1, 2022 12:29


          dont describe nested blocks

d6c8e9f


          Update CHANGELOG.md

a7075af


          Update run_test.go

a70f1b3

johakoch approved these changes

View reviewed changes

johakoch merged commit b536ef2 into master

johakoch deleted the beta-job branch

December 1, 2022 12:15

malud mentioned this pull request

fixup job: run on every interval as own routine #644

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet