Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GRU task to update DISTRI and NEEDLES_DIR from git if needed #5622

Merged
merged 2 commits into from
May 27, 2024

Conversation

asdil12
Copy link
Member

@asdil12 asdil12 commented May 6, 2024

lib/OpenQA/Schema/Result/ScheduledProducts.pm Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Utils.pm Outdated Show resolved Hide resolved
@asdil12 asdil12 force-pushed the git branch 3 times, most recently from 5044d28 to a4601bd Compare May 8, 2024 11:41
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
@asdil12 asdil12 force-pushed the git branch 3 times, most recently from 5b960ef to b229a5d Compare May 15, 2024 13:38
lib/OpenQA/Schema/Result/ScheduledProducts.pm Outdated Show resolved Hide resolved
lib/OpenQA/Schema/Result/ScheduledProducts.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
@asdil12 asdil12 force-pushed the git branch 8 times, most recently from 68d9f0f to 7c3b282 Compare May 15, 2024 15:30
@asdil12 asdil12 marked this pull request as ready for review May 15, 2024 15:30
@asdil12
Copy link
Member Author

asdil12 commented May 15, 2024

Still lacking tests (manually tested at least for the codepath via OpenQA/WebAPI/Controller/API/V1/Job.pm).

lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/WebAPI/Controller/API/V1/Job.pm Outdated Show resolved Hide resolved
lib/OpenQA/Schema/Result/ScheduledProducts.pm Outdated Show resolved Hide resolved
lib/OpenQA/Shared/Plugin/Gru.pm Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
okurz
okurz previously requested changes May 16, 2024
etc/openqa/openqa.ini Outdated Show resolved Hide resolved
@okurz
Copy link
Member

okurz commented May 16, 2024

@asdil12 you push changes but I don't know what they cover. Could you please write an update to state what your latest changes include?

@asdil12
Copy link
Member Author

asdil12 commented May 16, 2024

last push was the redundant config name.

@asdil12 asdil12 force-pushed the git branch 3 times, most recently from 1bb2d45 to 76f71a2 Compare May 21, 2024 10:46
@asdil12 asdil12 force-pushed the git branch 2 times, most recently from 0cbc44a to 2785898 Compare May 21, 2024 11:31
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Outdated Show resolved Hide resolved
lib/OpenQA/Task/Git/Clone.pm Show resolved Hide resolved
@asdil12 asdil12 force-pushed the git branch 4 times, most recently from 2938ee7 to e141f49 Compare May 21, 2024 13:52
etc/openqa/openqa.ini Outdated Show resolved Hide resolved
Copy link
Contributor

@Martchus Martchus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It generally looks good to merge now.

lib/OpenQA/WebAPI/Controller/API/V1/Job.pm Outdated Show resolved Hide resolved
@okurz
Copy link
Member

okurz commented May 21, 2024

There are still multiple unresolved suggestions, in particular about the die-messages

@asdil12
Copy link
Member Author

asdil12 commented May 22, 2024

There are still multiple unresolved suggestions, in particular about the die-messages

Not sure what you mean - all discussions are resolved.
If you think, anything is not resolved, please unresolve the according discussion.

I had a look at the die messages and they all look fine.

@asdil12 asdil12 force-pushed the git branch 2 times, most recently from 09ed682 to 566b796 Compare May 22, 2024 12:14
@kalikiana kalikiana dismissed okurz’s stale review May 23, 2024 08:50

I checked that all suggestions were taken into account

@Martchus
Copy link
Contributor

@asdil12 Looks like one of the tests is failing:

[12:16:50] t/14-grutasks.t ....... 30/? DBIx::Class::SQLMaker::ClassicExtensions::belch(): Warning: HASH ref as bind value in insert is not supported at /home/squamata/project/lib/OpenQA/Shared/Plugin/Gru.pm line 127
    # No tests run!

#   Failed test 'No tests run for subtest "handling failing GRU task"'
#   at t/14-grutasks.t line 584.

Copy link

codecov bot commented May 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.41%. Comparing base (dc3ed1b) to head (a38165b).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #5622    +/-   ##
========================================
  Coverage   98.40%   98.41%            
========================================
  Files         393      394     +1     
  Lines       38319    38438   +119     
========================================
+ Hits        37709    37828   +119     
  Misses        610      610            

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kalikiana
Copy link
Member

[13:50:56] t/full-stack.t ......... 33/?     # Unexpected Javascript console errors: [
    #   {
    #     level     => "SEVERE",
    #     message   => "http://localhost:9526/tests/Unable%20to%20read%20image:%20Can't%20open%20file%20%22/tmp/webui.worker-1.egZpq4BW/%22:%20No%20such%20file%20or%20directory%20at%20/home/squamata/project/lib/OpenQA/WebAPI/Controller/Running.pm%20line%20211. - Failed to load resource: the server responded with a status of 404 (Not Found)",
    #     source    => "network",
    #     timestamp => 1716558871922,
    #   },
    # ]
    # full result panel contents:
    # State: running, started less than a minute ago 
    #   
    # Scheduled product: job has not been created by posting an ISO
    # Assigned worker: 26686ca7dff7:1

    #   Failed test 'Expected result not found'
    #   at /home/squamata/project/t/lib/OpenQA/Test/FullstackUtils.pm line 76.

    #   Failed test 'job 8 is incomplete'
    #   at t/full-stack.t line 418.
    # 0

    #   Failed test 'reason for incomplete specified'
    #   at t/full-stack.t line 419.
    #                   'State: running, started less than a minute ago 
    #   
    # Scheduled product: job has not been created by posting an ISO
    # Assigned worker: 26686ca7dff7:1'
    #     doesn't match '(?^u:Failed to download.*non-existent.qcow2)'
[...]
#   Failed test 'Cache tests'
#   at t/full-stack.t line 442.
findElement: no such element: Unable to locate element: {"method":"css selector","selector":".embedded-logfile"} 

@kalikiana
Copy link
Member

Fullstack passed after a retry 🤔

codecov is still/again listing various gaps:

lib/OpenQA/Task/Git/Clone.pm#L24-L26
lib/OpenQA/Task/Git/Clone.pm#L29-L34
lib/OpenQA/Task/Git/Clone.pm#L37-L38
lib/OpenQA/Task/Git/Clone.pm#L41-L44
lib/OpenQA/Task/Git/Clone.pm#L48-L51
lib/OpenQA/Task/Git/Clone.pm#L54-L55
lib/OpenQA/Task/Git/Clone.pm#L58-L59
lib/OpenQA/Task/Git/Clone.pm#L61-L62
lib/OpenQA/Task/Git/Clone.pm#L65-L67
lib/OpenQA/Task/Git/Clone.pm#L70-L73
lib/OpenQA/Task/Git/Clone.pm#L76-L78
lib/OpenQA/Task/Git/Clone.pm#L81-L83
lib/OpenQA/Task/Git/Clone.pm#L86-L94
lib/OpenQA/Task/Git/Clone.pm#L96-L97
lib/OpenQA/Task/Git/Clone.pm#L99
lib/OpenQA/Task/Git/Clone.pm#L103-L106
lib/OpenQA/Task/Git/Clone.pm#L109-L110
lib/OpenQA/Task/Git/Clone.pm#L112-L113
lib/OpenQA/Task/Git/Clone.pm#L117
lib/OpenQA/Utils.pm#L613-L614

@asdil12 asdil12 force-pushed the git branch 2 times, most recently from 76b9f39 to db10a88 Compare May 27, 2024 15:35
Ticket: https://progress.opensuse.org/issues/159171

Co-authored-by: Oliver Kurz <okurz@suse.de>
Co-authored-by: Martchus <martchus@gmx.net>
Co-authored-by: Liv Dywan <liv@twotoasts.de>
@mergify mergify bot merged commit 8ed44a8 into os-autoinst:master May 27, 2024
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants