suse-edge/Factory
SHA256
17
0
Fork 17
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Make an effort to (partially) realign the Dockerfile with upstream #201

Merged
mchiappero merged 1 commits from mchiappero/Factory:alignment-dockerfile into main 2025-07-10 10:11:34 +02:00
Conversation 5 Commits 1 Files Changed 1 +3 -1
mchiappero commented 2025-06-30 18:11:54 +02:00
Owner
Copy Link

Apply misc changes to Dockerfile to align with upstream

Having a completely different image and build system the Dockerfile will
always differ quite significantly from the upstream one. Nonetheless,
it's still useful to make the commont parts to look alike as much as
possible to ease maintanance and update.

Note, this is just a small set of changes that are possible, more effort
into this direction may follow at later stage.

This PR depends on #200.

Apply misc changes to Dockerfile to align with upstream Having a completely different image and build system the Dockerfile will always differ quite significantly from the upstream one. Nonetheless, it's still useful to make the commont parts to look alike as much as possible to ease maintanance and update. Note, this is just a small set of changes that are possible, more effort into this direction may follow at later stage. This PR depends on #200.
mchiappero requested review from amorgante 2025-06-30 18:16:35 +02:00
mchiappero requested review from dprodanov 2025-06-30 18:16:49 +02:00
mchiappero requested review from nbelouin 2025-06-30 18:16:52 +02:00
mchiappero requested review from steven.hardy 2025-06-30 18:16:54 +02:00
mchiappero force-pushed alignment-dockerfile from fec5b28494 to b4e5833111 2025-07-02 15:36:58 +02:00 Compare
mchiappero force-pushed alignment-dockerfile from b4e5833111 to ecdb953890 2025-07-03 14:50:28 +02:00 Compare
mchiappero commented 2025-07-07 23:24:16 +02:00
Author
Owner
Copy Link

Tested, looks okay to me.

Tested, looks okay to me.
mchiappero force-pushed alignment-dockerfile from ecdb953890 to d00b6ece5f 2025-07-07 23:50:37 +02:00 Compare
mchiappero commented 2025-07-08 10:06:47 +02:00
Author
Owner
Copy Link

@nbelouin @steven.hardy Does it look okay to you?

@nbelouin @steven.hardy Does it look okay to you?
steven.hardy commented 2025-07-08 10:28:39 +02:00
Owner
Copy Link

lgtm - although in future we should perhaps consider an upstream change to make the location of the j2 templates more consistent - it's a bit weird atm since some end up in /tmp and others in /etc/ironic/ etc, not something to fix downstream though

lgtm - although in future we should perhaps consider an upstream change to make the location of the j2 templates more consistent - it's a bit weird atm since some end up in `/tmp` and others in `/etc/ironic/` etc, not something to fix downstream though
steven.hardy approved these changes 2025-07-08 10:28:51 +02:00
mchiappero commented 2025-07-08 13:09:14 +02:00
Author
Owner
Copy Link

lgtm - although in future we should perhaps consider an upstream change to make the location of the j2 templates more consistent - it's a bit weird atm since some end up in /tmp and others in /etc/ironic/ etc, not something to fix downstream though

I agree, had the same comment.

> lgtm - although in future we should perhaps consider an upstream change to make the location of the j2 templates more consistent - it's a bit weird atm since some end up in `/tmp` and others in `/etc/ironic/` etc, not something to fix downstream though I agree, had the same comment.
mchiappero merged commit d00b6ece5f into main 2025-07-08 14:07:19 +02:00
mchiappero deleted branch alignment-dockerfile 2025-07-08 14:07:20 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
amorgante
dprodanov
nbelouin
steven.hardy
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
amorgante antaloala atanasdinov atrendafilov dbekhit dprodanov eminguez fdegirmenci geoagriogiannis ipetrov117 jarndt jtomasek kzhelyazkov mchiappero mkrutov nbelouin roxenham steven.hardy
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#201
Delete Branch "mchiappero/Factory:alignment-dockerfile"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?