Skip to content

Conversation

@jbrown-xentity
Copy link
Contributor

Discussed with @FuhuXia on Slack.
These changes allow us to verify various jobs and processes before cutover, and keep things on auto-deployment to remove the need for a manual push.

When we cutover, we should eventually change all these catalog-next and catalog-beta references to catalog, but that should be a simple change (and we should scan the whole repo anyways, as various services will need to change).

@jbrown-xentity jbrown-xentity requested a review from a team September 4, 2025 14:38
@FuhuXia
Copy link
Member

FuhuXia commented Sep 4, 2025

  • catalog-next branch is falling behind main on secririty patch and other config changes. So we will need another effort to bring catalog-next security patch up-to-date by cherry-picking from main.
  • running sitemap generation on catalog-next-admin may end up with weird domain name in the sitemap files.
  • cron defined in non-default branch wont get run. so we need manual trigger the tasks, or run them in cf tasks, in order to achieve the goal to verify things before cutover.

FuhuXia
FuhuXia previously approved these changes Sep 4, 2025
@jbrown-xentity
Copy link
Contributor Author

Fixed the security patching if you want to re-review @FuhuXia .
True on the sitemap, although it will be accurate. We'll want to re-run that immediately after cutover.
True, cron jobs won't run. I wonder if there is a setting on that for github actions that allows other branches to run via the cron, or if that only works on the default? @neilmb can you check?

@jbrown-xentity jbrown-xentity merged commit 43f5170 into catalog-next Sep 4, 2025
12 checks passed
@jbrown-xentity jbrown-xentity deleted the catalog-next-automation branch September 4, 2025 18:03
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