Project

General

Profile

Actions
Copy link

action #98628

open

[easy][beginner] $cmd_srv_port exceed 65535

Added by dawei_pang over 3 years ago. Updated over 2 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
Feature requests
Target version:
QA (public) - future
Start date:
2021-09-14
Due date:
% Done:

0%

Estimated time:
Tags:
beginner, easy, entrance level

Description

Observation

Here is an example of worker configuration:

[40138]
HMC_HOSTNAME=hmc
HMC_MACHINE_NAME=ish414
WORKER_CLASS=hmc_ppc64le_lsh40138
LPAR_ID=138
SUT_IP=lsh40138
ARCH=ppc64le

In this example, the instance number of worker is 40138, base on the formula "QEMUPORT;integer;20002 + worker instance * 10" in doc/backend_vars.asciidoc, vars{QEMUPORT} = 421382, then $cmd_srv_port = 421383, the port is used for listening like "http://0.0.0.0:421383", when using script_run like 

testapi::assert_script_run(cmd="curl -f -v http://10.17.215.137:421383/FI1JZTNB0QGSIwjG/current_script > /tmp/scriptXT2eW.sh", timeout=90, quiet=undef, fail_message="")

curl will report:

  • Closing connection -1 curl: (3) URL using bad/illegal format or missing URL

Expectation: openQA should handle the situation when $cmd_srv_port larger than 65535, for example: set a random number as port.

Further details

entrance level issue

Workaround

Use openQA worker instance numbers well below 40000

Actions
Copy link
 #1

Updated by okurz over 3 years ago

  • Subject changed from $cmd_srv_port exceed 65535 to [easy][beginner] $cmd_srv_port exceed 65535
  • Description updated (diff)
  • Category set to Feature requests
  • Priority changed from Normal to Low
  • Target version set to future

thanks for the report. IMHO an "[easy][beginner]" task about missing basic boundary tests. I consider the impact very low. I added a workaround to the description and set the target for "low" prio in the "future" milestone, i.e. not to be followed up with by the SUSE QE Tools team anytime soon

Actions
Copy link
 #2

Updated by okurz over 2 years ago

  • Tags set to entrance level, easy, beginner
  • Description updated (diff)
Actions
Copy link

Also available in: Atom PDF