Skip to content
New issue

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

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

Already on GitHub? # to your account

Generate new initiatorname from scratch (Fixes: #14) #19

Merged
merged 1 commit into from
Nov 29, 2022

Conversation

dupondje
Copy link
Member

@dupondje dupondje commented Nov 23, 2022

Since the el9 packages don't build an initiatorname by default, we should not depend/use the file in factory anymore, but just generate the string ourselves.
Fixes: #14

@michalskrivanek
Copy link
Member

weird. let me try another PR...

@michalskrivanek
Copy link
Member

#15 worked ok...hm

@michalskrivanek
Copy link
Member

@lveyde any idea why it fails?

@dupondje
Copy link
Member Author

./src/imgbased/plugins/service.py:60:80: E501 line too long (87 > 79 characters)

Missed this one :)

@michalskrivanek
Copy link
Member

aaah, thanks!:)

Copy link
Member

@arachmani arachmani left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@lveyde lveyde left a comment

Choose a reason for hiding this comment

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

Please note that iscsi-iname itself generates a unique iSCSI node name on every invocation.

Thus it's output can be used directly, without the need to split it and then combine with the split uuid.

@dupondje dupondje requested a review from lveyde November 28, 2022 08:11
Since the el9 packages don't build an initiatorname by default, we
should not depend/use the file in factory anymore, but just generate the
string ourselves.

Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
@sandrobonazzola sandrobonazzola merged commit f346778 into oVirt:master Nov 29, 2022
@sandrobonazzola sandrobonazzola added this to the ovirt-4.5.4 milestone Dec 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

on almalinux 9.0 based node imgbased fails on missing file
5 participants