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

JCL Message class support #3340

Open
wants to merge 5 commits into
base: v2.x/staging
Choose a base branch
from

Conversation

LukasJaks
Copy link

Added message class for zwe start job. Message class parameter is being taken from configuration yaml.

Please check if your PR fulfills the following requirements. This is simply a reminder of what we are going to look for before merging your PR. If you don't know all of this information when you create this PR, don't worry. You can edit this template as you're working on it.

  • [ x] Tests for the changes have been added (for bug fixes / features)
  • [ x] Necessary documentation (if appropriate) have been added / updated
  • [ x] DCO signoffs have been added to all commits, including this PR

PR type

What type of changes does your PR introduce to Zowe? Put an x in the box that applies to this PR. If you're unsure about any of them, don't hesitate to ask.

  • Bugfix
  • [ x] Feature
  • Other... Please describe:

Relevant issues

Fixes

Changes proposed in this PR

Does this PR introduce a breaking change?

  • Yes
  • [ x] No

Does this PR do something the person installing Zowe should know about?


  • Affected function: general area of interest *
    Added field to zowe.yaml config file

  • Description: 1 line description *
    Added message class field into zowe configuration yaml file

  • Part: name of customizable file involved *
    zowe-example.yaml (schemas/zowe-yaml-schema.json)

multi-line description
Added message class field into zowe configuration file using zowe-yaml-schema.
Added code into start command to read zowe.job.msgclass value
and include "MSGCLASS={MSGCLASS}" in zowe JCL start job.

Is there a related doc issue or Pull Request?

Doc issue/PR number:

Other information

This is something we use and rely on in our company. If this is a rare thing that only very few systems would use, I would propose instead of zowe.job.msgclass, zowe.job.params where parameters other than msgclass could be explicitly set.

Signed-off-by: LukasJaks [email protected]

*Added message class for zwe start job. Message class parameter is being taken from configuration yaml

Signed-off-by: LukasJaks <[email protected]>
@1000TurquoisePogs
Copy link
Member

Is there an example you'd like to give within https://github.com/zowe/zowe-install-packaging/blob/v2.x/staging/example-zowe.yaml so that other users will see this option?

@LukasJaks
Copy link
Author

good point! in there now

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