Skip to content

Conversation

@ibmmqmet
Copy link
Collaborator

@ibmmqmet ibmmqmet commented Nov 20, 2018

Please ensure all items are complete before opening.

What

Add optional DIS QSTATUS metrics - which can be used on z/OS for minimal information

How

Add a queue.go following the pattern from channel.go, but had to modify names for uniqueness in package. Now use extended PCF command (_XR) format for z/OS compatibilty.

Testing

Used sample dashboards to collect data

@ibmmqmet ibmmqmet requested a review from parrobe November 20, 2018 07:51
mqcno->CCDTUrlPtr = NULL;
mqcno->CCDTUrlLength = length;
if (url != NULL) {
if (url != NULL && length > 0) {

This comment was marked as resolved.

case C.MQCFT_GROUP:
binary.Read(p, endian, &pcfParm.Parameter)
binary.Read(p, endian, &pcfParm.ParameterCount)

This comment was marked as resolved.

package mqmetric

/*
Copyright (c) IBM Corporation 2016, 2018
Copy link
Member

Choose a reason for hiding this comment

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

If this is a new file the copyright start date should be 2018 not 2016

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@parrobe parrobe merged commit 67a70ac into master Nov 20, 2018
@ibmmqmet ibmmqmet deleted the qstatus branch November 20, 2018 10:02
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