Project

General

Profile

action #34549

Updated by okurz almost 6 years ago

I need to retire pattern-minimalx-checked-boo1086058-20180320 and replace it with a correct one. 

 openQA does not provide a way to do that properly. I'd have to delete it manually from git which would break old test results. Would be better if there was a way to flag the needle so it's no longer used for new matches. 

 and more recent discussion about this: 

 ``` 
 [13 Jun 2018 09:11:03] <coolo> hands up! how broke tty2-selected needles? 
 [13 Jun 2018 09:12:09] <okurz> coolo: could be me, link? 
 [13 Jun 2018 09:12:29] <coolo> okurz: https://openqa.suse.de/group_overview/108 - spot the difference 
 [13 Jun 2018 09:12:44] <coolo>    delete mode 100644 tty2-selected-20151201.json 
 [13 Jun 2018 09:12:44] <coolo>    delete mode 100644 tty2-selected-20151201.png 
 [13 Jun 2018 09:12:49] <coolo> looks related :) 
 [13 Jun 2018 09:14:19] <okurz> coolo: I can't find the needle tty2-selected-20180613 I created over the webui 
 [13 Jun 2018 09:14:24] <coolo> so you didn't even do a MR for it? 
 [13 Jun 2018 09:14:58] <okurz> coolo: it's on osd. It's in a commit but that one has not been pushed 
 [13 Jun 2018 09:15:08] <okurz> commit df6b97935a364e341f58590dea75486ae99ba704 
 [13 Jun 2018 09:15:13] <coolo> because you broke the automatic pushing by doing manual commit at the same time  
 [13 Jun 2018 09:15:24] <coolo> anyway, if it's on osd it's broken - as breaking QAM 
 [13 Jun 2018 09:15:51] <okurz> aren't we always commiting "at the same time"? 
 [13 Jun 2018 09:16:35] <coolo> okurz: not so often from workstations 
 [13 Jun 2018 09:16:41] <okurz> https://openqa.suse.de/tests/1759358#step/first_boot/3 is where it's used properly. so I guess I broke tests because they were running at the time of merging my delete MR and new needles are only picked up after restarting … 
 [13 Jun 2018 09:16:42] <c4y> openQA: caasp-2.0-CaaSP-DVD-Incidents-x86_64-Build:7740:file.1528870658-QAM- 
 [13 Jun 2018 09:16:57] <okurz> hm 
 [13 Jun 2018 09:17:45] <coolo> but I give your theory a try and restart all fails 
 [13 Jun 2018 09:18:31] <okurz> ok, pushed as geekotest from osd as workaround. I guess this asks for a long-term feature to improve the handling of how needles reach the system. Can I expect you to handle that? or should I create a ticket with the chat log? 
 [13 Jun 2018 09:19:19] <coolo> okurz: I'm not following. if you created the needle before you deleted the old one, osd should never had a time where no tty2 needle existed 
 [13 Jun 2018 09:19:27] <coolo> so I assume you broke it by first deleting and then creating 
 [13 Jun 2018 09:20:15] <coolo> even though the timestamps of the git commits disagree 
 [13 Jun 2018 09:20:17] <coolo> but I don't care 
 [13 Jun 2018 09:23:17] <coolo> http://bugzilla.suse.com/show_bug.cgi?id=1086457#c54 - one more satisfied openqa user :)  
 [13 Jun 2018 09:23:17] <c4y> - 1086457: ltp_aiodio_part test failures: BTRFS critical (device vda2): unable to find logical 8820195328 - https://bugzilla.suse.com/show_bug.cgi?id=1086457 
 [13 Jun 2018 09:31:32] <okurz> coolo: Deleting a needle has an effect immediately:  
 [13 Jun 2018 09:31:32] <okurz> Could not open image /var/lib/openqa/cache/tests/sle/products/sle/needles/tty2-selected-20151201.png 
 [13 Jun 2018 09:31:32] <okurz> [2018-06-13T07:50:55.0141 CEST] [debug] SKIP(tty2-selected-20151201:missing PNG) 
 [13 Jun 2018 09:31:49] <okurz> in https://openqa.suse.de/tests/1759188/file/autoinst-log.txt but it should not have with proper caching to have a consistent set 
 [13 Jun 2018 09:32:11] <coolo> okurz: ah, I know 
 [13 Jun 2018 09:32:24] <okurz> let me pack that in a ticket 
 [13 Jun 2018 09:32:25] <coolo> w1 synced the old state and started job 
 [13 Jun 2018 09:32:33] <coolo> old needle was parsed in 
 [13 Jun 2018 09:32:41] <coolo> w2 synced the new stated 
 [13 Jun 2018 09:32:54] <coolo> w1 looked for his needle and didn't find it -> but ignored it 
 ```

Back