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

feat: add catalog sync script, used when catalog manifest isn't a template #774

Merged
merged 2 commits into from
Apr 17, 2024

Conversation

malept
Copy link
Member

@malept malept commented Apr 17, 2024

What this PR does / why we need it

Does what it says on the tin. This script only updates some basic fields, it's not exhaustive. An exhaustive updater would effectively require a Go-based script. It currently only supports Cortex but can be extended to other service catalogs' manifests.

Jira ID

DT-4324

@malept malept requested a review from a team as a code owner April 17, 2024 22:34

This comment was marked as resolved.

Copy link
Contributor

@yj-yan yj-yan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shell script to sync from various manifest files to cortex.yaml, looks legit.

@malept malept merged commit 9f13013 into main Apr 17, 2024
5 checks passed
@malept malept deleted the malept/chore/auto-update-cortex-manifest branch April 17, 2024 23:52
@getoutreach-ci-2
Copy link

🎉 This PR is included in version 2.27.0-rc.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

yj-yan added a commit that referenced this pull request May 20, 2024
* fix(stencil): upgrade modules (2024-04-16) (#768)

* fix(stencil): upgrade modules (2024-04-16)

* Update .circleci/config.yml

Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* Update package.json

Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* update yarn.lock

---------

Co-authored-by: getoutreach-ci-1[bot] <95656460+getoutreach-ci-1[bot]@users.noreply.github.com>
Co-authored-by: Phil Brocker <5331286+pbr0ck3r@users.noreply.github.com>
Co-authored-by: Mark Lee <malept@users.noreply.github.com>
Co-authored-by: Devbase CI <phil.brocker@outreach.io>

* chore: revert semantic-release dependency bumps (#769)

* chore: add cortex.yaml (#770)

* chore: add cortex.yaml

* remove opslevel

* fix(release-dryrun): CI-specific commands; use gh (#772)

* fix(release-dryrun): only setup git user name/email in CI
* refactor(release-dryrun): use gh for GitHub token

* feat: add `make stencil` and `make restencil` (#771)

* chore(deps): bump github.com/cloudflare/circl from 1.3.3 to 1.3.7 (#773)

Bumps [github.com/cloudflare/circl](https://github.com/cloudflare/circl) from 1.3.3 to 1.3.7.
- [Release notes](https://github.com/cloudflare/circl/releases)
- [Commits](cloudflare/circl@v1.3.3...v1.3.7)

---
updated-dependencies:
- dependency-name: github.com/cloudflare/circl
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: add catalog sync script, used when catalog manifest isn't a template (#774)

* feat(coverbot): update version of coverbot to v1.0.5 (#775)

* fix: bump kubeconform to 0.6.4 (#777)

* fix: replace vulnerable, unmaintained dependency with a fixed, maintained fork (#779)

* fix(go): parse HTTPS git URLs correctly (#780)

* feat(release): change the prerelease logic (#743)

* feat(release): change the prerelease logic

* fix: typo

* fix: linter

* fix: remove testing lines

* fix: add comments

* fix: add script and orb to trigger rc release

* fix: set branch to current if it is null

* fix: comments and remove testing lines

* fix: address comments

* fix: force to trigger release for testing access token

* fix: config release uer in service.yaml and default in box

* fix: revert testing

* fix: linter

* fix: box config for release user

* fix: use GH_TOKEN

* fix: allow partial match the commit message

* fix: skip release dryrun for empty change

* use gh auth

Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* update description

Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* fix format

---------

Co-authored-by: Devbase CI <devbase@outreach.io>
Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* chore(deps): bump github.com/getoutreach/localizer from 1.15.3 to 1.15.4 (#778)

* chore(deps): bump github.com/getoutreach/localizer from 1.15.3 to 1.15.4

Bumps [github.com/getoutreach/localizer](https://github.com/getoutreach/localizer) from 1.15.3 to 1.15.4.
- [Release notes](https://github.com/getoutreach/localizer/releases)
- [Changelog](https://github.com/getoutreach/localizer/blob/main/.releaserc.yaml)
- [Commits](getoutreach/localizer@v1.15.3...v1.15.4)

---
updated-dependencies:
- dependency-name: github.com/getoutreach/localizer
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Restencil

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: --global <mark.lee@outreach.io>
Co-authored-by: Mark Lee <malept@users.noreply.github.com>

* fix(linters): prettier 3+ log level flag compatibility (#781)

* fix: rc prerelease before unstable release (#783)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: getoutreach-ci-1[bot] <95656460+getoutreach-ci-1[bot]@users.noreply.github.com>
Co-authored-by: Phil Brocker <5331286+pbr0ck3r@users.noreply.github.com>
Co-authored-by: Mark Lee <malept@users.noreply.github.com>
Co-authored-by: Devbase CI <phil.brocker@outreach.io>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jan Pecek <jan.pecek@outreach.io>
Co-authored-by: Devbase CI <devbase@outreach.io>
Co-authored-by: --global <mark.lee@outreach.io>
@getoutreach-ci-1
Copy link
Contributor

🎉 This PR is included in version 2.27.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants