WIP: Various extensions for IBS #72

Draft
adrianSuSE wants to merge 25 commits from adrianSuSE/autogits:ibs_state into main
Member
No description provided.
adamm reviewed 2025-09-01 11:49:59 +02:00
@@ -821,6 +821,7 @@ func (e *GitHandlerImpl) GitSubmoduleList(gitPath, commitId string) (submoduleLi
for _, te := range tree.items {
if te.isTree() {
trees[p+te.name+"/"] = te.hash
submoduleList[p+te.name] = te.hash
Owner

This part I cannot merge because it will break the PR bot and other things.

This part I cannot merge because it will break the PR bot and other things.
adamm reviewed 2025-09-02 12:37:35 +02:00
@@ -241,3 +240,3 @@
for ; j < len(ref) && strings.Compare(results[i].Package, ref[j].Package) < 0; j++ {
}
if j < len(results) && results[i].Package == ref[j].Package {
Owner

the package check should probably stay here?

the package check should probably stay here?
adrianSuSE force-pushed ibs_state from e99a5d9b27 to a576c98f68 2025-09-02 14:07:08 +02:00 Compare
adrianSuSE force-pushed ibs_state from a576c98f68 to af6ccb205c 2025-09-09 09:04:03 +02:00 Compare
adrianSuSE added 1 commit 2025-09-23 09:29:56 +02:00
The parameter only takes a package name, but not a full subdirectory
string. So we hand over only the last part behind a / here
adrianSuSE added 1 commit 2025-09-24 10:41:24 +02:00
They need to keep access as they might need to be able to modify the
stage project. They could grant access anyway, by adding themselfs
as they own the upper project. No reason to force them the
extra trip or to hide build results first to them
adrianSuSE added 1 commit 2025-10-14 16:09:10 +02:00
They need to keep access as they might need to be able to modify the
stage project. They could grant access anyway, by adding themselfs
as they own the upper project. No reason to force them the
extra trip or to hide build results first to them
Owner

I'll set it to WIP as this is a pending patches branch, not so much merge-all branch. Most changes will will be cherry-picked instead of merged directly.

I'll set it to WIP as this is a pending patches branch, not so much merge-all branch. Most changes will will be cherry-picked instead of merged directly.
adamm changed title from Various extensions for IBS to WIP: Various extensions for IBS 2025-11-03 17:56:46 +01:00
adrianSuSE force-pushed ibs_state from 9c433bad6e to ebc34f9645 2026-01-13 14:06:06 +01:00 Compare
adrianSuSE force-pushed ibs_state from ebc34f9645 to a66393d6d0 2026-01-13 15:10:36 +01:00 Compare
adrianSuSE force-pushed ibs_state from a66393d6d0 to 7ddb1fcabc 2026-01-13 15:16:54 +01:00 Compare
adrianSuSE force-pushed ibs_state from 7ddb1fcabc to 6ffb596aba 2026-01-14 10:38:23 +01:00 Compare
adrianSuSE added 18 commits 2026-01-28 09:09:49 +01:00
Projects which are not using submodules as package directories
still need to be handled in stagings.
Esp. with multiple repositories:
* j counter was not reset
* wrong limit of results used (from reference not stage project)
* package name compare using == was always wrong (and unneeded here)
The parameter only takes a package name, but not a full subdirectory
string. So we hand over only the last part behind a / here
Any pull request with multiple packages was always successful, just when
one package  was successful build.

Dropping the "someSuccess" handling for now since it is currently
broken and I got also complaints that we anyway never should accept
broken submissions. Even when the failure did exist before.

This would anyway break an mainteannce/patchinfo build.

We may want to add a config option to allow the again under given
circumstances.
We must not use the upper directory name as part of the package name.
remove more code to ignore reference project entirely
Currently all our release managers insist anyway of only green states
eg. config changes only
it hit too many ones
This solves the issue of using the right credentials based on the
main repo.
Also it allows to rename the organisation.
This requires write permission by maintainer there
Gitea trims long titles so we need to compare if the trimmed length
is same, not entire string that will always differ.
to avoid merging pull requests where lfs objects are not correctly
registered. Can happen when user has not installed lfs for example.
Reviewed-on: atartamo/autogits#2
Merge remote-tracking branch 'antonello/ibs_state' into new_state
Some checks failed
go-generate-check / go-generate-check (pull_request) Failing after 27s
8f0f03e5a9
adrianSuSE added 3 commits 2026-01-29 08:55:02 +01:00
adrianSuSE force-pushed ibs_state from 23d06b9f6b to 2ce9cc455a 2026-02-02 11:48:23 +01:00 Compare
adrianSuSE force-pushed ibs_state from 2ce9cc455a to 2dd303a326 2026-02-02 14:48:43 +01:00 Compare
adrianSuSE force-pushed ibs_state from 2dd303a326 to 9bd5ed22fc 2026-02-02 14:50:33 +01:00 Compare
adrianSuSE force-pushed ibs_state from 9bd5ed22fc to 3256c056e7 2026-02-02 17:06:52 +01:00 Compare
adrianSuSE added 3 commits 2026-02-03 16:25:29 +01:00
* support project links
* ommit scmsync if empty
staging: Support using source from pullrequest in QA project
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 10s
db95345b91
A QA project can get configured to also rebuild the sources again based
on the binaries of other packages of the QA project.

This is esp. useful for bootstrap projects. We would keep the main
project in this case to ensure that the package itself is building
on current code base. But we would also test if it breaks other packages
or fail during bootstrap cycle.
adrianSuSE added 1 commit 2026-02-04 09:33:26 +01:00
staging: Protection against broken staging.config
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 9s
41932f5b38
when having multiple projects  with same name
adrianSuSE added 1 commit 2026-02-04 12:51:15 +01:00
staging: Set result to failed when we have missing packages
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 24s
3e08c7f310
in that case no build failure may appear so it was handled as success
adrianSuSE force-pushed ibs_state from 3e08c7f310 to 49ec5b7dcf 2026-02-04 12:51:42 +01:00 Compare
adrianSuSE added 1 commit 2026-02-04 14:29:06 +01:00
staging: Allow host names via env file
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 9s
c73c2e9417
or we can run it only for OBS via packages...
adrianSuSE added 2 commits 2026-02-05 10:11:53 +01:00
We need a fixed user for that
staging: Fix repeated QA project setups
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 23s
c30b837def
Don't touch an already setup project
adrianSuSE added 1 commit 2026-02-05 10:20:10 +01:00
staging: handle OBS API downtime
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 9s
7e31a5c843
don't write "Cannot fetch reference project meta" endless
adrianSuSE force-pushed ibs_state from 7e31a5c843 to 6c9509338a 2026-02-05 10:33:58 +01:00 Compare
adrianSuSE added 1 commit 2026-02-05 11:32:56 +01:00
staging: Adapt commit status link when QA fails
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
478317fa22
but main project was successful (eg. package builds, but appliance
fails, eg. due to broken deps)
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u ibs_state:adrianSuSE-ibs_state
git checkout adrianSuSE-ibs_state
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: git-workflow/autogits#72