[OS-BUILD PATCH 2/9] CI: Remove stale TAG and Makefile cruft

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus CI: Remove stale TAG and Makefile cruft The TAG variable turned into _TAG but the ci scripts were never updated to reflect that. No one noticed for over a year, therefore it must not be important. Also dist-merge-upstream had a branch check as a safety valve to prevent

[OS-BUILD PATCH 1/9] CI: Move os-build tracking to common area

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus CI: Move os-build tracking to common area Future changes will use also use os-build, move the tracking to a common area to reduce code duplication. Signed-off-by: Don Zickus diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index blahblah..blahblah 100644 --- a/.gitlab-ci.yml

[OS-BUILD PATCH 2/3] CI: Remove stale TAG and Makefile cruft

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus CI: Remove stale TAG and Makefile cruft The TAG variable turned into _TAG but the ci scripts were never updated to reflect that. No one noticed for over a year, therefore it must not be important. Also dist-merge-upstream had a branch check as a safety valve to prevent

[OS-BUILD PATCH 3/3] CI: Call script directly

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus CI: Call script directly Originally I thought it would be safer to use a Makefile to control the variables to the ci scripts. Now I realize I lose the ability to pass arguments unless I use environment variables. Let's call the script directly in preparation for use arguments

[OS-BUILD PATCH 0/3] CI: simple script cleanups part1

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2517 A collection of simple ci scripts cleanups in preparation for future automation and easy maintenance. Signed-off-by: Don Zickus --- redhat/Makefile | 12 ++-- .gitlab-ci.yml

[OS-BUILD PATCH 1/3] CI: Move os-build tracking to common area

2023-06-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus CI: Move os-build tracking to common area Future changes will use also use os-build, move the tracking to a common area to reduce code duplication. Signed-off-by: Don Zickus diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index blahblah..blahblah 100644 --- a/.gitlab-ci.yml

Re: [OS-BUILD PATCH] kernel.spec: remove "RPM_VMLINUX_H=$DevelDir/vmlinux.h" code chunk in %install

2023-06-08 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2458#note_1423778786 Yes, thanks for the explaination @dvlasenk ! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel

Re: [OS-BUILD PATCHv3] redhat/configs: aarch64: Turn on Display for OnePlus 6

2023-06-07 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2502#note_1422351094 @ecurtin - you would need the RHEL team to sign up to maintain it. And if it requires other config options too, you would also need those RHEL teams. I would poke @msalter, I believe

Re: [OS-BUILD PATCH] Enable IO_URING for RHEL

2023-05-10 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2472#note_1385284245 @ptalbert - curious, I believe io_uring is under drivers/block. I would have expected those maintainers to show up in the approval rules but they did not. Any thoughts on why? @jmoyer1

Re: [OS-BUILD PATCH] kernel.spec: remove "RPM_VMLINUX_H=$DevelDir/vmlinux.h" code chunk in %install

2023-05-02 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2458#note_1375015427 DevelDir I believe is a CKI env var? @veruu @mh21 ? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

Re: [OS-BUILD PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-05 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1342932614 I am ok with the thread being closed. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341364121 @prarit - ah ok. I misunderstood how the version worked. maximum rhel version wasn't obvious to me what that meant. It makes sense

Re: [OS-BUILD PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341362416 This is also something LEAP tests for and would catch if it is a problem. I am curious too. Though I thought fedora upgrades would have the same problem though? You update to latest

Re: [OS-BUILD PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341177765 Also by using RHEL_MAJOR you have packages kernel-redhat-0.1.x.el9 and kenrel- redhat-1.1.x.el10 which are basically identical other than configs. What are the customer expectations

Re: [OS-BUILD PATCHv2 0/5] redhat: introduce versioning for Gemini kernels

2023-04-04 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2386#note_1341172348 @jstancek - so for rhel-9.6 we can expect a kernel package name to be 'kernel- redhat-0.1.x.el9'? Does anyone intuitively trust a version of 0.X? Should we start at 1? Just a nitpick

Re: [OS-BUILD PATCH] Make RHJOBS container friendly

2023-03-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2375#note_1321278887 Hmm, as an end user it seems odd to have to know what cgroup version and other quirks are on the system to make correctly calculations. I would think that would be part of nproc's

[OS-BUILD PATCH] Make RHJOBS container friendly

2023-03-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus Make RHJOBS container friendly RHJOBS uses 'nproc --all' which is not friendly to process like containers that use taskset. Instead just use 'nproc'. Folks can override this with rhpkg.mk file if they choose. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b/redhat

[OS-BUILD PATCH] Remove scmversion from kernel.spec.template

2023-03-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove scmversion from kernel.spec.template The spec.template file was touching scmversion. The use case of scmversion was removed from upstream in 6.3. Let's remove it from our source tree. Signed-off-by: Don Zickus diff --git a/redhat/kernel.spec.template b/redhat

Re: [OS-BUILD PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1319161685 ugh nproc --all is wrong i believe ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1319106161 @mh21 - the man pages say nproc returns the number of cpus for the _process_ which works well for containers. Contrast this with /proc/cpuinfo which gives all available cpus

Re: [OS-BUILD PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1318986202 @mh21 - this will be single thread for cki unless cki uses 'make -j dist-srpm' (no number to default to nproc). ___ kernel mailing list

[OS-BUILD PATCHv3] Use RHJOBS for create-tarball

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus Use RHJOBS for create-tarball rpm-4.18.1 recently modified how _smp_mflags is defined and it breaks tarball creation: rpm --eval %{_smp_mflags} because the output is '-j${RPM_BUILD_NCPUS}' instead of '-j8'. While that is being worked on, let's simplify things and use what we

[OS-BUILD PATCHv2] Use _smp_build_nthreads for rpm eval

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus Use _smp_build_nthreads for rpm eval rpm-4.18.1 recently modified how _smp_mflags is defined and it breaks tarball creation: rpm --eval %{_smp_mflags} because the output is '-j${RPM_BUILD_NCPUS}' instead of '-j8'. While that is being worked on, let's simplify things and use

Re: [OS-BUILD PATCH] Use _smp_build_nthreads for rpm eval

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2370#note_1318936416 @jmflinuxtx - this should fix cki's problem. are you ok with tagging this in 'include in release'? @scweaver @mh21 - this works around the cki problem and is probably more correct

[OS-BUILD PATCH] Use _smp_build_nthreads for rpm eval

2023-03-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus Use _smp_build_nthreads for rpm eval rpm-4.18.1 recently modified how _smp_mflags is defined and it breaks tarball creation: rpm --eval %{_smp_mflags} because the output is '-j${RPM_BUILD_NCPUS}' instead of '-j8'. While that is being worked on, let's simplify things and use

Re: [OS-BUILD PATCH] redhat: adapt to upstream Makefile change

2023-03-01 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2344#note_1297508006 @clrkwllms - the only tricky part with this change is around the rebase. originally we put the change in a place that minimized the GA rebase merge conflicts. This may cause a conflict

[OS-BUILD PATCH] Remove cc lines from automatic configs

2023-02-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove cc lines from automatic configs The kernel-webhook is smarter and knows who the right reviewers are. The webhooks can also dynamically update them. Just remove the static cc assisgnment in the nightly automated config generation. Signed-off-by: Don Zickus diff --git

Re: [OS-BUILD PATCH 0/2] redhat/configs: rename ark config directory to rhel

2023-02-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2278#note_1276995815 git grep 'pending-ark' redhat/ shows redhat/scripts/genspec/gen_config_patches.sh Can you update that too? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCHv3 0/0] scriptlets: clean up declared installation dependencies, stop calling kernel-install in chroot or container

2023-02-06 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1617#note_1267368540 Closing, no activity. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2023-01-25 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1252887556 /me wears the cone of shame. Thanks @ptalbert @jwilsonrh ! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2023-01-25 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1252599534 @ptalbert @jwilsonrh - um help me unblock this thread? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2023-01-25 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1252596373 /unblock hehe. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org

Re: [OS-BUILD PATCHv3] makefile: Add -Werror support for RHEL

2023-01-06 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2240#note_1230246925 @prarit - hmm, ok I am torn here. I thank you for respecting the 'do not break ARK approach'. But then I wonder what is the value of this in ARK (besides the obvious don't lose changes

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1219424373 /block ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora

Re: [OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2236#note_1219422925 @hertonrk-rh - the revert is Frantisek's change. Not sure if you are familiar with it? I was wondering if we could replace 'CFLAGS=' with 'EXTRA_CFLAGS=' and have it still help

[OS-BUILD PATCHv2] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Revert "redhat: fix elf got hardening for vm tools" This reverts commit eb5aebd77015bc57616c5d05a7b6206c8ff871cb. A recent rework of the tools/vm area relies on a proper CFLAGS definition to include the proper header files. The kernel.spec file purposely overwrites

[OS-BUILD PATCH] Revert "redhat: fix elf got hardening for vm tools"

2022-12-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Revert "redhat: fix elf got hardening for vm tools" This reverts commit eb5aebd77015bc57616c5d05a7b6206c8ff871cb. A recent rework of the tools/vm area relies on a proper CFLAGS definition to include the proper header files. The kernel.spec file purposely overwrites

Re: [OS-BUILD PATCHv2] redhat/configs: Enable CRYPTO_CURVE25519 in ark

2022-12-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2225#note_1216392018 @jmflinuxtx - hmm, did we agree to avoid touching common and only let a bot do that? ___ kernel mailing list -- kernel@lists.fedoraproject.org

Re: [OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-12-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1203488108 created !2183 with 'touch' fix ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-12-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1203471910 ugh how dumb of me. I can create an MR that 'touch'es or 'rm -f'. Any preference @jmflinuxtx @ptalbert ? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH 0/2] redhat: Create a git repo when the srpm code is installed

2022-12-05 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2174#note_119735 Agreed. What problem are we solving here? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-11-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2165#note_1188537102 @jmflinuxtx - I believe this should catch the failed config MRs during the cronjob. ___ kernel mailing list -- kernel@lists.fedoraproject.org

[OS-BUILD PATCHv2] Check for kernel config git-push failures

2022-11-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus Check for kernel config git-push failures An automated nightly job is run to merge the upstream tree and check for new configs to be set. The new configs are automatically added to the redhat/configs/pending-ark directory and pushed. A second script then comes along and walks

[OS-BUILD PATCH] Check for kernel config git-push failures

2022-11-28 Thread Don Zickus (via Email Bridge)
From: Don Zickus Check for kernel config git-push failures An automated nightly job is run to merge the upstream tree and check for new configs to be set. The new configs are automatically added to the redhat/configs/pending-ark directory and pushed. A second script then comes along and walks

Re: [OS-BUILD PATCH] redhat/configs: enable CONFIG_TEST_LIVEPATCH=m for s390x

2022-11-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2160#note_1181763580 @prarit I think we should be ok with folks providing arch specific stuff. Towards GA we can evaluate_configs to move the configs higher if most arches support the option. Then run

Re: [OS-BUILD PATCH] Revert "Merge branch 'ark-make-help' into 'os-build'"

2022-11-21 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2158#note_1179935350 Unfortunate. Let's discuss this at our weekly meeting. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

Re: [OS-BUILD PATCHv2] [redhat] Update configs for the 9.2 DRM backport

2022-11-21 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2152#note_1179932973 @Conan_Kudo - the concern with that approach is, RHEL is not staffed to support all new technologies, we only support a small subset of what Fedora enables. We do not want to get our

Re: [OS-BUILD PATCH] Remove recommendation to use 'common' for config changes.

2022-11-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2157#note_1177768175 @jmflinuxtx @scweaver @prarit - here is my attempt to downplay the 'common' directory for configs. ___ kernel mailing list -- kernel

[OS-BUILD PATCH] Remove recommendation to use 'common' for config changes.

2022-11-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove recommendation to use 'common' for config changes. Config changes should be focused on the 'ark' or 'fedora' directory. Update the docs to promote that. Signed-off-by: Don Zickus diff --git a/redhat/docs/submitting-contributions.rst b/redhat/docs/submitting

Re: [OS-BUILD PATCH] redhat: avoid picking up stray editor backups when processing configs

2022-11-15 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2149#note_1172697891 @prarit - I don't think it will because we are running a 'cat' command not a 'git' command. ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH] redhat: override `make help` to include dist-help

2022-11-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2144#note_1171547161 @prarit @jtoppins_redhat - this change should fix it -ifneq (,$(findstring n,$(MAKEFLAGS))) +ifneq (,$(findstring n,$(filter-out --%,$(MAKEFLAGS

Re: [OS-BUILD PATCH] redhat: override `make help` to include dist-help

2022-11-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2144#note_1171535058 @jtoppins_redhat - found it. https://gitlab.com/cki-project/kernel- ark/-/blob/os-build/redhat/Makefile#L22-26 You are being bitten by @prarit RHTEST checker. It looks for a '-n

Re: [OS-BUILD PATCH] redhat: override `make help` to include dist-help

2022-11-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2144#note_1171514727 Oddly commenting that out, allows the dist-configs-check target to run and successfully generate the kernel-*.configs that are failing to copy over. Still investigating

Re: [OS-BUILD PATCHv3] redhat: Force remove tmp file

2022-11-08 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2135#note_1164129535 Silly nitpick, seeing that BTF is a default technology for RH, it wasn't obvious to me which arch/variant would _not_ have it enabled as I thought that was _the_ bug. Once I figured out

Re: [OS-BUILD PATCH] redhat/configs: Disable CONFIG_FW_LOADER_COMPRESS

2022-10-24 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2112#note_1146743250 @hertonrk-rh - you had to enable this for rhel-9 it appears. can you provide some background? ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCH 0/8] redhat: Cleanup scripts

2022-10-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2095#note_1139862125 @jwilsonrh - +1. directories should have a 'theme'. I am ok with 'scripts/gating'. or 'scripts/osci' as those scripts are specific to osci

Re: [OS-BUILD PATCH 0/3] [redhat] enable some key Tegra SoC features

2022-10-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2083#note_1138460406 @ahs3 - we don't see any code using this config. Am I missing something (other than the defconfig file)? ___ kernel mailing list -- kernel

[OS-BUILD PATCH] Remove unused ci scripts

2022-10-11 Thread Don Zickus (via Email Bridge)
From: Don Zickus Remove unused ci scripts The ARK CI process has changed over time and the following scripts are no longer used. Remove them to avoid confusion for future contributors. * ark-rebase-patches.sh used to rebase patches for ark-patches. ark-patches was replaced by the current

Re: [OS-BUILD PATCHv2] redhat/configs: Disable fbdev drivers and use simpledrm everywhere

2022-10-03 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1533#note_1122556452 @Lyude - do you have time for a quick review? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel

Re: [OS-BUILD PATCH 0/2] Reduce disk space requirements of the kernel build

2022-09-26 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1950#note_1114646949 @jwilsonrh - can you reply here? ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCHv5 0/9] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067980884 I am happy if Justin is happy. Thanks @hertonrk-rh ! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067930412 Perfect example env! So a couple of things. I believe the 'rebase' word is incorrect and the 'Resolves' line should be removed?? (I thought it was with your sed magic?). Sadly, I

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067842595 Thanks! I think this series is good prep work for the real fix, which is to correctly handle the marker != last_marker case (merge update). This patchset will just say 'rebase', which

Re: [OS-BUILD PATCHv4 0/8] ark: rework changelog generation process during dist-release

2022-08-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1980#note_1067725269 Thanks @hertonrk-rh for this series! It looks to be an improvement in the overall workflow (which shows with the later commits removing workarounds). I don't have any objections

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-08-15 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_1065198628 Closing in favor of https://gitlab.com/cki-project/kernel- ark/-/merge_requests/1900 ___ kernel mailing list -- kernel@lists.fedoraproject.org

Re: [OS-BUILD PATCH] redhat/Makefile: Clean linux tarballs

2022-08-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1975#note_1056132350 Hmm. long ago, creating the linux tarball took a couple of minutes. So if you were constantly running make dist-srpm and friends that couple of minutes got annoying (much like how dist

Re: [OS-BUILD PATCH] Use new HOME path in ssh command

2022-06-23 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1887#note_1002826830 with containers!412 merged, this MR is unnecessary. Closing. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send

[OS-BUILD PATCH] Use new HOME path in ssh command

2022-06-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Use new HOME path in ssh command CKI recently updated the HOME variable to point to /cki instead of /root. The .gitlab-ci.yml file applies all ssh tweaks to ~/.ssh/ which is /cki/.ssh. However, ssh still uses /root/.ssh. So all the tweaks are ignored and ssh connections

[OS-BUILD PATCHv3] Test: Broken code, do not merge

2022-06-22 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

Re: [OS-BUILD PATCH 0/2] create-data.sh: Reduce specfile data output

2022-06-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1878#note_990962307 Thank you! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org

[OS-BUILD PATCHv2] Test: Broken code, do not merge

2022-06-14 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

[OS-BUILD PATCH] Enable CKI on os-build MRs only

2022-06-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus Enable CKI on os-build MRs only Not all the old Fedora branches have the latest Makefile and specfile changes to support the self-test check. Disable the Fedora branches for now. Signed-off-by: Don Zickus diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index blahblah..blahblah

[OS-BUILD PATCH] Test: Broken code, do not merge

2022-06-13 Thread Don Zickus (via Email Bridge)
From: Don Zickus Test: Broken code, do not merge diff --git a/redhat/docs/index.rst b/redhat/docs/index.rst index blahblah..blahblah 100644 --- a/redhat/docs/index.rst +++ b/redhat/docs/index.rst @@ -117,7 +117,7 @@ Contributor Guide Maintainer Guide -.. toctree

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: remove stray *.hardlink-temporary files, if any

2022-05-27 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1811#note_962630004 It does. Though I have a patch to reduce it down to changes to the %%SPEC lines, which is more realistic. I have to wait for @prarit to come back from PTO to review it. But for this MR

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: remove stray *.hardlink-temporary files, if any

2022-05-27 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1811#note_962560082 @ptalbert @jmflinuxtx @prarit - so this breaks the self-tests and if included will break self-tests for all future configs. Hopefully this doesn't get merged until that is resolved

Re: [OS-BUILD PATCHv2 0/4] redhat/self-tests: Add self-tests to gitlab-ci.yaml

2022-05-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1741#note_954969180 I don't know who or what was originally blocking this thread, but I believe it to be resolved (at least Prarit and Veronika appear to think things are good). So I am clicking the resolve

Re: [OS-BUILD PATCHv2 0/4] redhat/self-tests: Add self-tests to gitlab-ci.yaml

2022-05-20 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1741#note_954971065 @jmflinuxtx - up to you if you want to sneak this in before Monday's merge window opening. The last commit in this series is the piece that enables the cki part (run_self_test: true

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951732268 Oh and @omos filed this https://gitlab.com/cki-project/kernel- ark/-/merge_requests/1781 because he got hit with my buildreqs change from last week. That MR kinda undoes the check

Re: [OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951723323 Sure, but it doesn't describe why some rpmbuild commands get it, while others do not. Hence the -binary part. If you are building binaries you needed the extra suffix. This makes

Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951720936 @hertonrk-rh @prarit @jmflinuxtx To answer a variety of questions. Yes srpm processes the configs. It kinda has to otherwise what is rpmbuild going to use to build the kernel? :-) Why

Re: [OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951705198 I can't prove it mathematically, but in order to build the SRPM, you need to build the configs. What does dist-configs-check do that regular dist-configs does not? And if it is missing

[OS-BUILD PATCH 2/3] [redhat] Makefile: drop duplicate dist-sources deps

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Ondrej Mosnacek [redhat] Makefile: drop duplicate dist-sources deps do-rpmbuild itself already depends on dist-sources, thus no need to specify when we depend on do-rpmbuild already. Signed-off-by: Ondrej Mosnacek diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah

[OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus [redhat] Explicitly set srpm requirements It isn't clear what packages are necessary to be installed to generate the srpm for the kernel. Use another spec macro to call out the exact packages needed. This allows a makefile target to be used to verify the right packages have

[OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus [redhat] Add dist-buildreq-check for binary rpm builds By simplifying the package check to srpm only, we neglect to check the packaging for local binary rpm building. Add that Makefile dependency for those cases. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b

[OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786 Developers want to build a srpm locally and submit it to a build server like Koji. The current process checks the BuildRequires in the kernel.spec file to ensure all the right packages

Re: [OS-BUILD PATCH 0/2] Makefile: only depend on dist-buildreq-check when building actual RPMs

2022-05-17 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1781#note_949736929 I used a macro to address this. This seems to work. Thoughts? https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786 ___ kernel

Re: [OS-BUILD PATCH 0/2] Makefile: only depend on dist-buildreq-check when building actual RPMs

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1781#note_948589022 @hertonrk-rh agreed. There are srpm deps: https://cki- project.gitlab.io/kernel-ark/#cloning-the-repository Now I will agree that dist-buildreq-check might have swung to far in one

Re: [OS-BUILD PATCHv2 0/17] redhat/Makefile: General improvements and fixes

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1757#note_948427545 It definitely depends on the repo. I think I had so more garbage that it inflated things. 'git gc' removed enough junk that my 'git log --oneline' is down to 12. Looking into this more

Re: [OS-BUILD PATCHv2 0/17] redhat/Makefile: General improvements and fixes

2022-05-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1757#note_948374003 maybe it is just me, but a 'git log --oneline' shows 13 chars already. git log --oneline --abbrev=8|grep -Ev "^[0-9a-f]{,10} " shows me 9 commits that are 11 chars l

[OS-BUILD PATCH] Fix nightly merge CI

2022-05-12 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix nightly merge CI My recent MR for builddeps broke the nightly merge CI scripts because the script was using the wrong supported container. Update the CI script to use the correct supported CKI container image for ARK. Signed-off-by: Don Zickus diff --git a/.gitlab

[OS-BUILD PATCHv2] Add system-sb-certs for RHEL-9

2022-05-10 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add system-sb-certs for RHEL-9 Copy the RHEL-9 signing requirement over to ARK. This change is wrapped with "{?rhel}{?centos} && !{?eln}" to allow Fedora and ELN to continue using their signing technology. The system-sb-certs is just a metapackage for ei

Re: [OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-10 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1774#note_941572465 Hmm, I remembered centos-stream, but forgot about centos. I will push that update. Not sure if that change or just using '!{?fedora}' is easier

Re: [OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1774#note_940375865 @hertonrk-rh @jmflinuxtx - is this a respectful way to wrap this buildreq? ___ kernel mailing list -- kernel@lists.fedoraproject.org

[OS-BUILD PATCH] Add system-sb-certs for RHEL-9

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus Add system-sb-certs for RHEL-9 Copy the RHEL-9 signing requirement over to ARK. This change is wrapped with "{?rhel} && !{?eln}" to allow Fedora and ELN to continue using their signing technology. The system-sb-certs is just a metapackage for either rhel-sb-

[OS-BUILD PATCH] Fix dist-buildcheck-reqs

2022-05-09 Thread Don Zickus (via Email Bridge)
From: Don Zickus Fix dist-buildcheck-reqs The check for rpmspec buildrequires wasn't working because of a Makefile typo. Verified by deleting known buildreqs. Signed-off-by: Don Zickus diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah 100644 --- a/redhat/Makefile +++ b

Re: [OS-BUILD PATCH] redhat: configs: Disable TPM 1.2 specific drivers

2022-04-08 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1713#note_906356838 @jmflinuxtx - hehe a good use for test case under the selftest work Prarit is doing. :-) ___ kernel mailing list -- kernel

Re: [OS-BUILD PATCHv5 0/8] redhat/self-test: Add new tests

2022-03-31 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1706#note_896519247 we can add it later. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: Fix intel-speed-select compile

2022-03-31 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1721#note_896031021 I know jolsa had similar issues a couple of years ago with perf. He put some interesting workarounds in the spec file to deal with them, "perf_make". Is that something we

Re: [OS-BUILD PATCHv5 0/8] redhat/self-test: Add new tests

2022-03-30 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1706#note_894990774 block ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora

Re: [OS-BUILD PATCHv5 0/8] redhat/self-test: Add new tests

2022-03-30 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1706#note_894990301 @jmflinuxtx - do you want a linux-stable version number in the self-tests too? Or is that covered by the 'official' example and never breaks

<    1   2   3   4   5   6   7   8   >