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

[Enabler] [module_utils/jobs.py] Update job.py to obtain class using zoau over regex #1722

Open
1 task done
ddimatos opened this issue Sep 25, 2024 · 0 comments · May be fixed by #1751
Open
1 task done

[Enabler] [module_utils/jobs.py] Update job.py to obtain class using zoau over regex #1722

ddimatos opened this issue Sep 25, 2024 · 0 comments · May be fixed by #1751
Assignees
Labels
Enabler Enabler task

Comments

@ddimatos
Copy link
Collaborator

Is there an existing issue for this?

  • There are no existing issues.

Enabler description

In jobs.py, there is some pattern matching happening to extract the jobs CLASS, this is not necessary and can be done with ZOAU. ZOAU jls does support jobclass, see the doc here

Additionally, these fields are appended when using the -l option:
	<jobclass> <serviceclass> <priority> <ASID> <creationdate> <creationtime> <queueposition> <programname>

Code in question doing matching:

 if len(job["class"]) < 1:
                        if "- CLASS " in tmpcont:
                            tmptext = tmpcont.split("- CLASS ")[1]
                            job["class"] = tmptext.split(" ")[0]

Ansible module

No response

@ddimatos ddimatos added the Enabler Enabler task label Sep 25, 2024
@ddimatos ddimatos added the Needs Triage Issue need assessment by a team member(s) label Sep 25, 2024
@AndreMarcel99 AndreMarcel99 removed the Needs Triage Issue need assessment by a team member(s) label Oct 16, 2024
@AndreMarcel99 AndreMarcel99 linked a pull request Oct 17, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enabler Enabler task
Projects
Status: 🔍 Validation
Development

Successfully merging a pull request may close this issue.

3 participants