[basic_discovery.py] passing --max-pub-ops results error

[doronbl]

Confirm by changing [ ] to [x] below to ensure that it's a bug:

• I've searched for previous similar issues and didn't find any solution

Known Issue

• I'm using ATS data type endpoint: the endpoint should look like <prefix>-ats.iot.<region>.amazonaws.com

Describe the bug
I'm running basic_discovery.py with custom --max-pub-ops value (e.g. --max-pub-ops 20) and getting this error:

Connected!
Traceback (most recent call last):
  File "basic_discovery.py", line 109, in <module>
    while loop_count < args.max_pub_ops:
TypeError: '<' not supported between instances of 'int' and 'str'

SDK version number
V2

Platform/OS/Device
uname -a:
Linux ip-10-0-2-49 5.11.0-1022-aws #23~20.04.1-Ubuntu SMP Mon Nov 15 14:04:48 UTC 2021 aarch64 aarch64 aarch64 GNU/Linux

Graviton2 ec2 instance emulating the device

To Reproduce (observed behavior)
Run basic_discovery.py and add --max-pub-ops 20 to the list of parameters

Expected behavior
Message loop shpould publish 20 messages

Logs/output

Connected!
Traceback (most recent call last):
  File "basic_discovery.py", line 109, in <module>
    while loop_count < args.max_pub_ops:
TypeError: '<' not supported between instances of 'int' and 'str'

REMEMBER TO SANITIZE YOUR PERSONAL INFO

awscrt.io.init_logging(awscrt.io.LogLevel.Warn, 'stderr')

Additional context
Add any other context about the problem here.

[jmklix]

Recently the command line args have been standardized to use underscores. Can you make sure you have the latest version of the sdk and then change the arg to --max_pub_ops.

[github-actions]

Greetings! It looks like this issue hasn’t been active in longer than a week. We encourage you to check if this is still an issue in the latest release. Because it has been longer than a week since the last update on this, and in the absence of more information, we will be closing this issue soon. If you find that this is still a problem, please feel free to provide a comment or add an upvote to prevent automatic closure, or if the issue is already closed, please feel free to open a new one.