Skip to content

Increase Starlark execution limit, abstract limit resolver#1

Closed
colindean wants to merge 298 commits intomasterfrom
bump-starlark-limit
Closed

Increase Starlark execution limit, abstract limit resolver#1
colindean wants to merge 298 commits intomasterfrom
bump-starlark-limit

Conversation

@colindean
Copy link
Copy Markdown
Owner

renovate Bot and others added 30 commits April 29, 2022 08:18
…r rulesets (go-vela#630)

* add edit to scm/webhook

* populate event action field and use for rules if provided

* get rid of event action setter for tag event - mistake

* handle all types of edited actions

Co-authored-by: Easton Crupper <eastoncrupper@Eastons-MacBook-Air.local>
…a#622)

* stash apply

* handle webhook redelivery

* fix merge conflict fallout

Co-authored-by: David May <1301201+wass3r@users.noreply.github.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
wass3rw3rk and others added 27 commits July 6, 2023 16:31
…using claims (go-vela#901)

* fix(api/schedule): make validateEntry more strict and set updated_by using claims

* rm docker compose update

* add test cases to validateEntry
* fix(schedules): ignore trigger for first time schedule

* fix(schedules): determine trigger off current UTC time

* chore: save work

* cleanup: ignore inactive schedules

* feat: add interval for schedules

* chore: address slack feedback

* chore: fix typos

* fix: processing timed schedules

* fix: processing schedules

* fix: typo in comment

* chore: address review feedback

* temp: add test docker compose

* fix: finalize

* revert: add test docker compose

---------

Co-authored-by: Easton Crupper <65553218+ecrupper@users.noreply.github.com>
Co-authored-by: David May <49894298+wass3rw3rk@users.noreply.github.com>
go-vela#902)

Co-authored-by: dave vader <48764154+plyr4@users.noreply.github.com>
* fix(deps): update deps (patch)

* fix(deps): update deps (patch)

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ecrupper <easton.crupper12@gmail.com>
…ela#907)

* chore(release): upgrade types to v0.20.0 + gorm and starlark upgrade

* update Dockerfile to pin to alpine version for certs

* go get correct docker version syntax

* every day I surprise myself with my foolishness

* pin gorm to an actual tag
…templates (go-vela#916)

* fix(compiler): convert local templates to an engine field for nested templates

* add WithLocalTemplates test
* chore(release): upgrade types to v0.20.1

* other deps
* enhance: getting visbility to match on vela and SCM

* fix: modifying test cases to consider new visibility changes

---------

Co-authored-by: Claire.Nicholas <Z00B3R3@w6nxmk4t6t.target.com>
* refactor(db): return schedule on create and update

* fix integration tests

---------

Co-authored-by: David May <49894298+wass3rw3rk@users.noreply.github.com>
* enhance: add context to Repos

* chore: add WithContext test
)

* init commit

* adding db tests

* compiled to build itinerary

* add mock

* itinerary -> executable

* fix mock

* update some extra db configs

* pull in updated types

* encrypt and decrypt

* Update database/executable/interface.go

Co-authored-by: dave vader <48764154+plyr4@users.noreply.github.com>

* pesky linter

* add integration test for executables

* try this

* with encryption resource opt

* Update database/executable/opts.go

Co-authored-by: dave vader <48764154+plyr4@users.noreply.github.com>

---------

Co-authored-by: dave vader <48764154+plyr4@users.noreply.github.com>
Co-authored-by: David May <1301201+wass3r@users.noreply.github.com>
* enhance: adding a branch field to scheduled builds

* enhance: adding a branch field to scheduled builds

* enhance: adding a branch field to scheduled builds

* enhance: fixing merge conflicts for branch in scheduled builds

* enhance: fixing merge conflicts for branch in scheduled builds

---------

Co-authored-by: Claire.Nicholas <Z00B3R3@w6nxmk4t6t.hq.target.com>
Co-authored-by: Kelly Merrick <kelly.merrick@target.com>
Co-authored-by: Claire.Nicholas <Z00B3R3@w6nxmk4t6t.target.com>
Co-authored-by: David May <1301201+wass3r@users.noreply.github.com>
5,000 was too few to enable the example added to the testdata to work, so was 6,000. I chose 7,500 arbitrarily after a test at 10,000 and both worked.

In the long term, this should probably be configurable so as not to require recompilation. For now, this kicks the can down the road while allowing this build matrix use case to exist.
@colindean colindean closed this Aug 28, 2023
@colindean colindean deleted the bump-starlark-limit branch August 28, 2023 20:21
@colindean colindean restored the bump-starlark-limit branch August 28, 2023 20:21
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.