-
Couldn't load subscription status.
- Fork 15.1k
[en] discussions about automated-tasks-with-cron-jobs.md #34465
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
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can refer to man page too.
ceec747 to
9799369
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, I'd like to see a bigger revision. This page is due a cleanup (see issue #4500). @windsonsea maybe we, Kubernetes SIG Docs, can help you make that bigger cleanup happen?
If we're revising this page, I recommend also putting in a note about the risk of setting .spec.concurrencyPolicy to Allow (which is the default): if your Job is still running after the next one is due to start, you can end up consuming lots of resources.
That can either harm availability or your wallet, depending on whether you autoscale your cluster.
setting .spec.concurrencyPolicy to Forbid or Replace can be a wiser choice in many cases.
970e430 to
9e75c60
Compare
|
@tengqm haha, did I solve this rebase issue by the following tries, right? 😺 PS G:\Project\k8s\k8s> history
Id CommandLine
-- -----------
17 git checkout main
18 git fetch upstream
19 git remote add upstream GIT_REPO
20 git remote -v
21 git fetch upstream
22 git remote add upstream https://github.com/windsonsea/k8s.git
23 git remote set-url upstream https://github.com/windsonsea/k8s.git
24 git fetch upstream
25 git rebase upstream/main
26 git push
27 git checkout docsite3
28 git log
29 git rebase -i main
30 git status
31 git rebase --continue
32 git add
33 git add .
34 git status
35 git rebase -i HEAD~10
36 git rebase --continue
37 git status
38 git commit --amend --no-edit
39 git push origin docsite3 -f |
|
@windsonsea You only need to do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's good to go.
/lgtm
Remove squash label since it's already squashed into one commit.
/remove-label tide/merge-method-squash
|
LGTM label has been added. Git tree hash: 0b2047f22787bb82963799f0474193b1095485f4
|
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kbhawkey The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The path is content/en/docs/tasks/job/automated-tasks-with-cron-jobs
Added tags to each title for easily referenced by other pages and other minor changes.