Project

General

Profile

action #126623

Tags can't coexist with builds that contain colons size:M

Added by szarate 2 months ago. Updated 24 days ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Concrete Bugs
Target version:
Start date:
2023-03-24
Due date:
% Done:

0%

Estimated time:
Difficulty:
Tags:

Description

Observation

If a build has the following format: 2023-03-24 14:33:39+01:00 when creating a tag, the webui doesn't allow the tagging

date --rfc-3339=seconds
2023-03-24 14:33:39+01:00

Expected result:

  1. A message is returned to the user when the syntax is invalid
  2. If BUILD can be anything, tag/labels should allow the user to use i.e tag:"2023-03-24 14:33:39+01:00":likethis either via webui or via api

Acceptance Criteria

  • AC1: Tags not matching builds yield clear user feedback
  • AC2: Tags with special characters like : can be used if quoted
  • AC3: Intended, not-matching build tags can still be created, e.g. for a build that will come in some hours

Suggestions

  • Colons are common in build names in some use cases and should be allowed, provide a mean to escape colons
  • Compare to the yellow highlighting we use for labels, highlight for tags that already matches at least one present build

History

#1 Updated by okurz 2 months ago

  • Target version set to Ready

#2 Updated by cdywan 2 months ago

  • Subject changed from Tags can't coexist with builds that contain semicolons to Tags can't coexist with builds that contain semicolons size:M
  • Description updated (diff)
  • Status changed from New to Workable

#3 Updated by okurz 2 months ago

  • Priority changed from Normal to Low

#4 Updated by osukup about 1 month ago

  • Assignee set to osukup

#5 Updated by osukup about 1 month ago

  • Subject changed from Tags can't coexist with builds that contain semicolons size:M to Tags can't coexist with builds that contain colons size:M
  • Status changed from Workable to In Progress

problem with colon is caused by usage of colon as delimiter :(

https://github.com/os-autoinst/openQA/blob/master/lib/OpenQA/Schema/Result/Comments.pm#L127-L138

#7 Updated by osukup 24 days ago

  • Status changed from In Progress to Resolved

merged,

Also available in: Atom PDF