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

Re-align the directory structure with the upstream one #195

Closed
mchiappero wants to merge 1 commits from mchiappero:upstream-file-sync into main
pull from: mchiappero:upstream-file-sync
merge into: suse-edge:main
Conversation 7 Commits 1 Files Changed 21 +11 -10
mchiappero commented 2025-06-25 10:58:12 +02:00
Owner
Copy Link

In order to update the repo to the latest content upstream, restore the original repository structure and leverage similar common steps in the Dockerfile.

In order to update the repo to the latest content upstream, restore the original repository structure and leverage similar common steps in the Dockerfile.
steven.hardy commented 2025-06-25 10:59:56 +02:00
Owner
Copy Link

Does this actually work in OBS? When I last rebased I needed to stick with the flat-files because OBS couldn't handle the subdirectories, but I'm perhaps not recalling the details - @nbelouin any thoughts?

Does this actually work in OBS? When I last rebased I needed to stick with the flat-files because OBS couldn't handle the subdirectories, but I'm perhaps not recalling the details - @nbelouin any thoughts?
steven.hardy requested review from nbelouin 2025-06-25 11:00:34 +02:00
nbelouin commented 2025-06-25 11:01:33 +02:00
Owner
Copy Link

We use directories for helm charts successfully, so I think we may be able to use the same approach here, the OBS build will tell us if it doesn't work anyway

We use directories for helm charts successfully, so I think we may be able to use the same approach here, the OBS build will tell us if it doesn't work anyway
mchiappero commented 2025-06-25 11:32:06 +02:00
Author
Owner
Copy Link

That's (mainly) why it's WIP.

That's (mainly) why it's WIP.
👍 1
steven.hardy commented 2025-06-25 12:28:07 +02:00
Owner
Copy Link

Another option could be to package the scripts as an RPM which we then install via the Dockerfile, that would also make it easier if we want to carry patches e.g a spec file with some list of patches we apply to an unmodified upstream tarball

Another option could be to package the scripts as an RPM which we then install via the Dockerfile, that would also make it easier if we want to carry patches e.g a spec file with some list of patches we apply to an unmodified upstream tarball
mchiappero force-pushed upstream-file-sync from 4be6bc21b8 to df10224c8d 2025-06-26 23:41:58 +02:00 Compare
mchiappero force-pushed upstream-file-sync from df10224c8d to c7bec0de7d 2025-06-27 17:10:04 +02:00 Compare
mchiappero changed title from WIP: Re-align the directory structure with the upstream one to Re-align the directory structure with the upstream one 2025-06-30 11:36:09 +02:00
mchiappero requested review from dprodanov 2025-06-30 11:36:48 +02:00
mchiappero requested review from amorgante 2025-06-30 11:37:01 +02:00
mchiappero requested review from steven.hardy 2025-06-30 11:37:02 +02:00
mchiappero commented 2025-06-30 11:37:41 +02:00
Author
Owner
Copy Link

I'd say we can merge it, any other update will follow this restore structure.

I'd say we can merge it, any other update will follow this restore structure.
mchiappero closed this pull request 2025-06-30 16:50:21 +02:00
steven.hardy commented 2025-06-30 17:37:51 +02:00
Owner
Copy Link

@mchiappero not had time to review this yet, did you mean to close it?

@mchiappero not had time to review this yet, did you mean to close it?
mchiappero commented 2025-06-30 17:51:22 +02:00
Author
Owner
Copy Link

@mchiappero not had time to review this yet, did you mean to close it?

I meant to merge it, but I'm going to push a set of new PR with proper ordering soon. Please approve the new PR once open as permissions are a problem but the directory structure is not. Thank you!

> @mchiappero not had time to review this yet, did you mean to close it? I meant to merge it, but I'm going to push a set of new PR with proper ordering soon. Please approve the new PR once open as permissions are a problem but the directory structure is not. Thank you!
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 32s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -1m8s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
nbelouin
dprodanov
amorgante
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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#195
Delete Branch "mchiappero:upstream-file-sync"

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?