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

Update retention policy settings #2004

Conversation

Oluwadaminiola
Copy link
Collaborator

No description provided.

@danvixent danvixent changed the base branch from main to daniel/feat/move_retention_policy_to_instance_cfg May 20, 2024 11:38
@danvixent danvixent marked this pull request as ready for review May 20, 2024 11:38
@danvixent danvixent merged commit 427bc60 into daniel/feat/move_retention_policy_to_instance_cfg May 20, 2024
4 checks passed
@danvixent danvixent deleted the pelumi/update/retention-policy-setting branch May 20, 2024 11:38
danvixent added a commit that referenced this pull request Jul 4, 2024
* fix: move retention policy to instance config

* Update retention policy settings (#2004)

* remove retention policy settings from project config

* update config settings with retention policy setting

* fix:
- remove api migrations
- add docs
- fix fetchConfiguration

* fix: fix test syntac errors

* fix: add stub

* fix: remove non-existent fields from query

* fix: fix TestProjectService_CreateProject

* fix: fix config tests

* fix build error

* update project form with search policy

* remove search policy from config form

* fix: add retention cli flags

* fix: define retention cli flags

* fix: fix cli flag not overriding

* fix: change cron spec

* fix: change cron spec

* fix: change cron spec

* fix: check for len(projects)

* fix: change cron spec

* fix:
- remove GetRetentionPolicyConfig
- fix errors in UpdateProject

* fix: remove duplicate import

* fix: add project.Config.SearchPolicy to params in UpdateProject

* fix: skip if no record was exported

* fix: check len projects in sub loader

* fix: web package-lock.json

* fix: remove go.uber.org/automaxprocs pkg

* fix: check for len(projects)

* fix: add gzip compression

* fix: dont  update config singleton with the instance id if command is bootstrap

* fix: remove compressWriter

* fix: remove .gz

* fix: skip limiter if limit or duration is zero

* fix: add version to skip config load

* fix:
- remove build envs from build.sh
- set retention policy cronspec to original value

* fix: add skipConfigLoadCmd

---------

Co-authored-by: Pelumi Muyiwa-Oni <Pelumioni25@gmail.com>
Oluwadaminiola added a commit that referenced this pull request Oct 1, 2024
* fix: move retention policy to instance config

* Update retention policy settings (#2004)

* remove retention policy settings from project config

* update config settings with retention policy setting

* fix:
- remove api migrations
- add docs
- fix fetchConfiguration

* fix: fix test syntac errors

* fix: add stub

* fix: remove non-existent fields from query

* fix: fix TestProjectService_CreateProject

* fix: fix config tests

* fix build error

* update project form with search policy

* remove search policy from config form

* fix: add retention cli flags

* fix: define retention cli flags

* fix: fix cli flag not overriding

* fix: change cron spec

* fix: change cron spec

* fix: change cron spec

* fix: check for len(projects)

* fix: change cron spec

* fix:
- remove GetRetentionPolicyConfig
- fix errors in UpdateProject

* fix: remove duplicate import

* fix: add project.Config.SearchPolicy to params in UpdateProject

* fix: skip if no record was exported

* fix: check len projects in sub loader

* fix: web package-lock.json

* fix: remove go.uber.org/automaxprocs pkg

* fix: check for len(projects)

* fix: add gzip compression

* fix: dont  update config singleton with the instance id if command is bootstrap

* fix: remove compressWriter

* fix: remove .gz

* fix: skip limiter if limit or duration is zero

* fix: add version to skip config load

* fix:
- remove build envs from build.sh
- set retention policy cronspec to original value

* fix: add skipConfigLoadCmd

---------

Co-authored-by: Pelumi Muyiwa-Oni <Pelumioni25@gmail.com>
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.

2 participants