action #58319

[openqa-client] make --form visible in the help output

Added by ybonatakis 6 months ago. Updated 5 months ago.

Status:ResolvedStart date:17/10/2019
Priority:HighDue date:
Assignee:mkittler% Done:

0%

Category:Feature requests
Target version:Done
Difficulty:
Duration:

Description

openqa-client has a hidden parameter, the --form parameter, which is necessary for some calls. it should be in the help output with a proper description

i try to make a API call to job_templates_scheduling/num:id using post with openqa-client and because of the big size of the yaml file passed to the template, it returns 414 - Request-URI Too Long.

For example the following command

openqa-client --host http://somehost.suse.cz --apikey xxx --apisecret yyy job_templates_scheduling/1 post --form schema=JobTemplates-01.yaml preview=0 template="$(cat JobGroups/SLE_15_YaST.yaml)"

without the --form is failing with Error 414 - Request-URI Too Long
because JobGroups/SLE_15_YaST.yaml is too big. Although with a small yaml file the call works even without the parameter, but it doesnt seem to be the right approach.

History

#1 Updated by coolo 5 months ago

  • Priority changed from Normal to High
  • Target version set to Ready

We should have a --file parameter if we want yaml to be uploaded

#2 Updated by coolo 5 months ago

(and document both --file and --form)

#3 Updated by mkittler 5 months ago

  • Assignee set to mkittler

#4 Updated by mkittler 5 months ago

  • Status changed from New to In Progress

PR: https://github.com/os-autoinst/openQA/pull/2478

Note that there's no --file but there's --params to read parameters from a JSON file. It is already documented but likely not useful to post YAML because one needed to prepare a JSON file like {"template":"products: {}\nscenarios: {}"}.

#5 Updated by mkittler 5 months ago

  • Target version changed from Ready to Current Sprint

#6 Updated by mkittler 5 months ago

  • Status changed from In Progress to Resolved
  • Target version changed from Current Sprint to Done

PR has been merged

Also available in: Atom PDF