Skip to content

Update command outputs to k8s 1.13 based#12771

Merged
k8s-ci-robot merged 6 commits intokubernetes:masterfrom
makocchi-git:fix/cronjob
Mar 12, 2019
Merged

Update command outputs to k8s 1.13 based#12771
k8s-ci-robot merged 6 commits intokubernetes:masterfrom
makocchi-git:fix/cronjob

Conversation

@makocchi-git
Copy link
Copy Markdown
Contributor

No description provided.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 22, 2019
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. language/en Issues or PRs related to English language labels Feb 22, 2019
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit 59f7ed0

https://deploy-preview-12771--kubernetes-io-master-staging.netlify.com

@makocchi-git
Copy link
Copy Markdown
Contributor Author

preview is here
original

Copy link
Copy Markdown
Contributor

@Rajakavitha1 Rajakavitha1 left a comment

Choose a reason for hiding this comment

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

You should see that the cron job "hello" successfully scheduled a job at the time specified in LAST-SCHEDULE.
There are currently 0 active jobs, meaning that the job has completed or failed.

There is no specified time under LAST SCHEDULE in the output. I see only 50s. Also LAST-SCHEDULE is hyphenated?

Copy link
Copy Markdown
Contributor

@Rajakavitha1 Rajakavitha1 left a comment

Choose a reason for hiding this comment

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

Mon Aug 29 21:34:09 UTC 2016
Hello from the Kubernetes cluster

Is this the output you see in k8s 1.13 as well? May be you can update the actual date and time you see in 1.13

@Rajakavitha1
Copy link
Copy Markdown
Contributor

Rajakavitha1 commented Feb 22, 2019

Thanks for your PR @makocchi-git !!!! I checked the updates on https://deploy-preview-12771--kubernetes-io-master-staging.netlify.com/docs/tasks/job/automated-tasks-with-cron-jobs/

I just made some suggestions based on the changes you have made.
I do not have a 1.13 version environment to check the output so may be I will assign it to someone who can validate it after you made the updates.

Copy link
Copy Markdown
Contributor

@Rajakavitha1 Rajakavitha1 left a comment

Choose a reason for hiding this comment

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

Because this is a task and there are steps. I suggest the following outline:

  1. Download the example corn job
  2. Check the status of the corn job
  3. Watch the job that is being created
  4. View the scheduled job
  5. View the pods that the last scheduled job created
    a. Check the pod name
    b. View the pod log

@Rajakavitha1
Copy link
Copy Markdown
Contributor

/assign @chenopis

@Rajakavitha1
Copy link
Copy Markdown
Contributor

@chenopis Please review this PR for the output in 1.13.

@makocchi-git
Copy link
Copy Markdown
Contributor Author

@Rajakavitha1 Thank you for your comments.

I fixed following:

  • Add downloading yaml process
  • LAST-SCHEDULE to LAST SCHEDULE
  • Update datetime in a pod log (k8s 1.13 is not born yet in 2016...😉)

@zparnold
Copy link
Copy Markdown
Contributor

@makocchi-git One slight nit, then it looks good to go

Co-Authored-By: makocchi-git <makocchi@gmail.com>
@makocchi-git
Copy link
Copy Markdown
Contributor Author

@zparnold Thank you for your review!

@makocchi-git
Copy link
Copy Markdown
Contributor Author

@chenopis ping

@DanyC97
Copy link
Copy Markdown
Contributor

DanyC97 commented Mar 12, 2019

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 12, 2019
@cody-clark
Copy link
Copy Markdown
Contributor

Thanks for the PR, @makocchi-git! This is great!

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cody-clark

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 12, 2019
@k8s-ci-robot k8s-ci-robot merged commit 23dacbb into kubernetes:master Mar 12, 2019
yagonobre pushed a commit to yagonobre/website that referenced this pull request Mar 14, 2019
* Update command outputs to k8s 1.13 based

* Update command outputs to k8s 1.13 based (2)

* Add the way to download the example manifest and change datetime in a pod log

* create a job from remote url

* Update content/en/docs/tasks/job/automated-tasks-with-cron-jobs.md

Co-Authored-By: makocchi-git <makocchi@gmail.com>
@makocchi-git makocchi-git deleted the fix/cronjob branch June 18, 2020 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants