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

refactor slug for cronjobs #102

Merged
merged 1 commit into from
Jul 19, 2024
Merged

refactor slug for cronjobs #102

merged 1 commit into from
Jul 19, 2024

Conversation

matthyx
Copy link
Contributor

@matthyx matthyx commented Jul 12, 2024

No description provided.

@matthyx matthyx changed the title refactor instanceidhandler with unique container type refactor slug for cronjobs Jul 12, 2024
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

1 similar comment
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

@matthyx matthyx force-pushed the slug branch 2 times, most recently from 3b1dff5 to 94704fb Compare July 12, 2024 17:03
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

// if the Pod is created by a CronJob, its parent is a Job named after the CronJob
// with the scheduled timestamp appended to it (unix time in minutes).
// https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/cronjob/utils.go#L277
if ownerReference.Kind == "Job" {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

here is the magic part for CronJobs

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

@matthyx matthyx marked this pull request as ready for review July 19, 2024 05:31
@matthyx matthyx merged commit 7552a40 into main Jul 19, 2024
7 of 8 checks passed
@matthyx matthyx deleted the slug branch July 19, 2024 06:40
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