Project

General

Profile

Actions

Bug #63672

closed

qa: nothing provides lua-devel needed by ceph-2:18.0.0-7530.g67eb1cb4.el8.x86_64

Added by Patrick Donnelly 5 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Urgent
Target version:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):


Related issues 2 (1 open1 closed)

Has duplicate rgw - Bug #63788: Cephadm tests fail from "nothing provides lua-devel needed by ceph-2:19.0.0-44.g2d90d175.el8.x86_64"DuplicateCasey Bodley

Actions
Blocks rgw-testing - Backport #63639: reef: lua integration testsNewYuval LifshitzActions
Actions #1

Updated by Yuval Lifshitz 5 months ago

in the above test, lua-devel is found for:
Actions #2

Updated by Casey Bodley 5 months ago

are we positive that lua-devel is a runtime dependency? did you try without and see failures? i don't understand why luarocks only lists lua as a dependency if it needs -devel also

Actions #3

Updated by Casey Bodley 5 months ago

Actions #4

Updated by Casey Bodley 5 months ago

  • Blocks deleted (Bug #63616: lua integration tests)
Actions #5

Updated by Casey Bodley 5 months ago

Actions #6

Updated by Yuval Lifshitz 5 months ago

are we positive that lua-devel is a runtime dependency?

luarocks compile the packages, so it needs lua.h which comes with lua-devel

did you try without and see failures?

yes

i don't understand why luarocks only lists lua as a dependency if it needs -devel also

because it is not luarocks itself that depends on that. it what it does with the packages
luarocks is pure lua code, just need lua. but when it compiles a lua package that has C code in it (not all of them has), it needs lua.h

Actions #7

Updated by Casey Bodley 5 months ago

Zack is trying to get this repo enabled in https://github.com/ceph/ceph-cm-ansible/pull/749

Actions #8

Updated by Casey Bodley 5 months ago

the #sepia folks haven't figured out how to enable this repo yet. if that can't be resolved in a timely manner, we could prepare a temporary revert for the lua-devel dependency. that would break the rgw/lua tests, but unblock testing on rhel

Actions #9

Updated by Yuval Lifshitz 5 months ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 54789
Actions #10

Updated by Casey Bodley 5 months ago

  • Status changed from Fix Under Review to Resolved
Actions #11

Updated by Casey Bodley 5 months ago

from #sepia thread:

Dan Mick: so I think I finally enabled the repo for rhel8 hosts
Dan Mick: the "proper" way to fix this would be to recapture the fog image; I'll work on that next

Actions #12

Updated by Casey Bodley 5 months ago

  • Has duplicate Bug #63788: Cephadm tests fail from "nothing provides lua-devel needed by ceph-2:19.0.0-44.g2d90d175.el8.x86_64" added
Actions #13

Updated by Casey Bodley 5 months ago

another update from #sepia slack:

rhel8.6 jobs should work now, and allow installing lua-devel

Actions #14

Updated by Yuval Lifshitz 5 months ago

as lab issue is now fixed, the lua-devel runtime dependency is added back: https://github.com/ceph/ceph/pull/54935

Actions

Also available in: Atom PDF