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.

Sign up for GitHub

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

Jump to bottom

Fixes 5052: refactor jobs and temp/repo association #906

Merged
merged 4 commits into from
Nov 26, 2024
Merged

Fixes 5052: refactor jobs and temp/repo association #906

merged 4 commits into from
Nov 26, 2024

Conversation

jlsherrill
Copy link
Member

@jlsherrill jlsherrill commented Nov 25, 2024 •
edited
Loading

Summary

This pulls 2 refactors out of #865

  • refactors jobs into a single command ./cmd/jobs/main.go
  • removes old unused jobs
  • refactors the template_repository_configurations association into an official model

Testing steps

  1. Tests pass
  2. running go run cmd/jobs/main.go retry_failed_task seems to run the appropriate code

@jlsherrill
Copy link
Member Author

https://issues.redhat.com/browse/HMS-5052

@xbhouse xbhouse mentioned this pull request Nov 26, 2024
Open
@jlsherrill jlsherrill merged commit 7fd9a08 into content-services:main Nov 26, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xbhouse xbhouse xbhouse approved these changes

Assignees
No one assigned
Labels
no-qe-needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jlsherrill @xbhouse