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

[FBZ-10492] Fix Custom Cron for DB and Util instances #133

Merged
merged 3 commits into from Oct 26, 2022

Conversation

kdotbautista
Copy link
Contributor

Description of your patch

Fix Custom Cron for DB and Util instances

Recommended Release Notes

Fixed Custom Cron for DB and Util instances

Estimated risk

Low

Components involved

All clients that uses custom-cron recipe

Dependencies

  • custom-cron

Description of testing done

  • Spin up a v7 environment with App Master, DB Master and 2 Util instances named as test1 and test2
  • Add custom-cron recipe with custom crons below:
default["custom_crons"] = [{name: "test1", time: "10 * * * *", command: "echo 'test1'", instance_name: "test1"},
                          {name: "db", time: "10 1 * * *", command: "echo 'testdb'", instance_name: "db"},
                          {name: "app_master", time: "10 1 * * *", command: "echo 'testapp_master'", instance_name: "app_master"},
                          {name: "util", time: "10 1 * * *", command: "echo 'testutil'", instance_name: "util"}]
  • Apply custom recipe
  • Check the instances if the cron jobs were created for the ff.
    • app_master cron job for the App Master
    • db cron job for the DB Master
    • test1 and util cron jobs for the Utility instance named as test1
    • util cron job for the Utility instance named as test2

QA Instructions

  • Spin up a v7 environment with App Master, DB Master and 2 Util instances named as test1 and test2
  • Add custom-cron recipe with custom crons below:
default["custom_crons"] = [{name: "test1", time: "10 * * * *", command: "echo 'test1'", instance_name: "test1"},
                          {name: "db", time: "10 1 * * *", command: "echo 'testdb'", instance_name: "db"},
                          {name: "app_master", time: "10 1 * * *", command: "echo 'testapp_master'", instance_name: "app_master"},
                          {name: "util", time: "10 1 * * *", command: "echo 'testutil'", instance_name: "util"}]
  • Apply custom recipe
  • Check the instances if the cron jobs were created for the ff.
    • app_master cron job for the App Master
    • db cron job for the DB Master
    • test1 and util cron jobs for the Utility instance named as test1
    • util cron job for the Utility instance named as test2

@kdotbautista kdotbautista self-assigned this Oct 26, 2022
@kdotbautista kdotbautista changed the title [FBZ-10492] - Fix Custom Cron for DB and Util instances [FBZ-10492] Fix Custom Cron for DB and Util instances Oct 26, 2022
@rupikakapoor
Copy link
Contributor

except that I have tested that recipe works fine.

Copy link
Contributor

@mushyy mushyy left a comment

Choose a reason for hiding this comment

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

LGTM

@mushyy mushyy merged commit 2a714b2 into next-release Oct 26, 2022
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.

None yet

3 participants