Project

General

Profile

Actions

action #124230

closed

coordination #80142: [saga][epic] Scale out: Redundant/load-balancing deployments of openQA, easy containers, containers on kubernetes

coordination #92854: [epic] limit overload of openQA webUI by heavy requests

action #97190: Limit size of initial requests everywhere, e.g. /, /tests, etc., over webUI and API

"Off-by-one" error when using the limit=1 openQA API parameter

Added by okurz over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2023-02-09
Due date:
% Done:

0%

Estimated time:

Description

Observation

With a query like

openqa-cli api --host https://openqa.opensuse.org jobs version=Tumbleweed scope=relevant arch='x86_64' flavor=NET test='minimalx' latest=1 limit=1 | jq '.jobs | .[] | .settings.BUILD'

I could find out the latest build where this test is available but despite the parameter limit=1 I actually get two results instead of the expected one result:

"20230207"
"20230208"

Acceptance criteria

  • AC1: API route calls with limit=1 give back just one entry, not multiple

Suggestions

  • Look into what has been done so far in #97190
  • Crosscheck for "off-by-one" errors in the implementation
Actions #1

Updated by mkittler over 1 year ago

  • Assignee set to mkittler
Actions #2

Updated by mkittler over 1 year ago

  • Status changed from New to Feedback
Actions #3

Updated by mkittler over 1 year ago

  • Status changed from Feedback to Resolved

The PR has been merged and deployed. The cmd from the ticket description works now as expected.

Actions

Also available in: Atom PDF