Skip to content

Hotfix k8s bq updater#41

Open
monoid-is-a-monad wants to merge 4 commits into
releasedfrom
hotfix-k8s-bq-updater
Open

Hotfix k8s bq updater#41
monoid-is-a-monad wants to merge 4 commits into
releasedfrom
hotfix-k8s-bq-updater

Conversation

@monoid-is-a-monad

Copy link
Copy Markdown
Contributor

Mitigates:
#37

Notes:
Cronjobs are enabled so when we deploy this, cronjobs will start to run
There are 2 secrets intentially missing

@monoid-is-a-monad monoid-is-a-monad requested a review from CodiePP May 27, 2026 17:23

@CodiePP CodiePP left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

just a minor thing: we can use kustomization to concretize the image

restartPolicy: Never
containers:
- name: cardano-bq-slot-sync
image: code.blockchain-applied.com/bca/cardano-bq-updater:1.0.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
image: code.blockchain-applied.com/bca/cardano-bq-updater:1.0.0
image: cardano-bq-updater

- cronjob.yaml

images:
- name: code.blockchain-applied.com/bca/cardano-bq-updater

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- name: code.blockchain-applied.com/bca/cardano-bq-updater
- name: cardano-bq-updater
newName: code.blockchain-applied.com/bca/cardano-bq-updater

app: cardano-bq-slot-sync
spec:
suspend: true
schedule: "3,33 * * * *"

@CodiePP CodiePP Jun 2, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we should only run it once per hour at hh:34

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

and, it would be easier if we use the timezone UTC:

  timeZone: Etc/UTC

app: cardano-bq-epoch-sync
spec:
suspend: true
schedule: "16 5 * * *"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this currently runs at 01:10 UTC.

and, it would be easier if we use the timezone UTC:

  timeZone: Etc/UTC

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