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

Add minimal JiraJob code #15

Merged
merged 3 commits into from
Mar 21, 2024
Merged

Add minimal JiraJob code #15

merged 3 commits into from
Mar 21, 2024

Conversation

kkaarreell
Copy link
Collaborator

@kkaarreell kkaarreell commented Mar 15, 2024

This PR adds minimal code for JiraJob class and associated classes Issue and Recipe.

@kkaarreell kkaarreell requested a review from happz March 15, 2024 11:11
@kkaarreell kkaarreell changed the title Support job_recipe in component-config.yaml [WIP] Support job_recipe in component-config.yaml Mar 15, 2024
@kkaarreell
Copy link
Collaborator Author

kkaarreell commented Mar 15, 2024

$ REQUESTS_CA_BUNDLE=/etc/pki/tls/cert.pem newa event jira
produces few jira- files like this one

$ cat jira-128049-RHEL-8.10.0-NEWA-12.yaml
erratum:
  builds: []
  release: RHEL-8.10.0
event:
  id: '128049'
  type_: erratum
jira:
  id: NEWA-12
recipe:
  url: https://path/to/recipe.yaml

The new keys here are jira and recipe. This file is currently created only when job_recipe is defined in the configuration.

@kkaarreell kkaarreell changed the title [WIP] Support job_recipe in component-config.yaml Initial JiraJob template Mar 15, 2024
@kkaarreell kkaarreell changed the title Initial JiraJob template Add minimal JiraJob code Mar 15, 2024
newa/cli.py Outdated Show resolved Hide resolved
newa/__init__.py Outdated Show resolved Hide resolved
newa/__init__.py Outdated Show resolved Hide resolved
@kkaarreell kkaarreell requested a review from happz March 19, 2024 07:36
@thrix thrix merged commit 722a985 into main Mar 21, 2024
1 check passed
@thrix thrix deleted the ks_job_recipe branch March 21, 2024 12:50
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.

3 participants