Packit on `ublue-os/packages`

yeah there might be something with packit-cli
82 Replies
tulip🌷
tulip🌷•2mo ago
ehrm what the sigma ok so oh nice we might be able to just use that then on the gh actions
Zeglius
ZegliusOP•2mo ago
Reproduce CI environment locally | Packit
This used to be a question in our FAQ and
Zeglius
ZegliusOP•2mo ago
But maybe there is a way to trigger remote builds
tulip🌷
tulip🌷•2mo ago
lets just use the hosted stuff they have im guessing packit-cli lets us trigger the builds
Zeglius
ZegliusOP•2mo ago
:huh: So what i said
tulip🌷
tulip🌷•2mo ago
real didnt read it
Zeglius
ZegliusOP•2mo ago
just packit-shell and elts start digging
tulip🌷
tulip🌷•2mo ago
oh there yo go packit build in-copr oh it absolutely works
Zeglius
ZegliusOP•2mo ago
? Does it show in github or something?
tulip🌷
tulip🌷•2mo ago
packit build in-copr --package aurora failed tho
Zeglius
ZegliusOP•2mo ago
mmm
No description
tulip🌷
tulip🌷•2mo ago
oh we also might want to add a packit lint thing then on packit config change why is this god damn thing failing tho
Zeglius
ZegliusOP•2mo ago
Something during the rebase yep, work in my backup branch
tulip🌷
tulip🌷•2mo ago
wondeful 😭 copy over ur packit config from the backup branch then just commit as a new one LOL
Zeglius
ZegliusOP•2mo ago
by any chance do you have git rerere disabled?
tulip🌷
tulip🌷•2mo ago
yup
Zeglius
ZegliusOP•2mo ago
:v
tulip🌷
tulip🌷•2mo ago
no idea what that is
Zeglius
ZegliusOP•2mo ago
It remembers rebase conflict solutions Have a video but lets focus on this first
tulip🌷
tulip🌷•2mo ago
oh i do i have it rerere.enabled=true huh do you mind pushing ur functioning config to the remote branch? i wanna test it out
Zeglius
ZegliusOP•2mo ago
its on my packit-efficient-trigger-bak branch
tulip🌷
tulip🌷•2mo ago
git checkout .packit.yml packit-efficient-trigger-bak i think
Zeglius
ZegliusOP•2mo ago
oh
tulip🌷
tulip🌷•2mo ago
from the non-bak branch
Zeglius
ZegliusOP•2mo ago
k did, but 0 changes wait now works
tulip🌷
tulip🌷•2mo ago
whats ur command line
Zeglius
ZegliusOP•2mo ago
No description
Zeglius
ZegliusOP•2mo ago
ok so no this only triggers locally
tulip🌷
tulip🌷•2mo ago
yeah damn ooooh
tulip🌷
tulip🌷•2mo ago
in-copr | Packit
Submit a Copr build of the present content in the upstream repository.
tulip🌷
tulip🌷•2mo ago
lets just go with the god damn comments for now fuck it
Zeglius
ZegliusOP•2mo ago
mmmThat requires us storing the token as a secret
tulip🌷
tulip🌷•2mo ago
yeah fuck that
Zeglius
ZegliusOP•2mo ago
gonna take a break and keep going with the comment thing
tulip🌷
tulip🌷•2mo ago
so what are we missing with the comment stuff? i can work on it while ur on the break oh um i think the workflow isnt running cuz you dont have branch perms on the thing maybe i can PR it from the ublue-os/packages repo itself
Zeglius
ZegliusOP•2mo ago
mmm I think is almost done
tulip🌷
tulip🌷•2mo ago
GitHub
feat: add packit with automated building by tulilirockz · Pull Req...
This is @Zeglius 's PR #337 but on ublue-os/package's github context so we can run the workflows on the incoming PR
Zeglius
ZegliusOP•2mo ago
? Why a separate pr?
tulip🌷
tulip🌷•2mo ago
incoming PRs from other contexts need some funky shit to run like new workflows due to security reasons (you dont want people just adding a workflow then fucking adding a crypto miner on ur actions or something) i just made a new PR from a branch on the ublue-os/packages repo so the new workflow will run i can just rebase mine when you make a new change then i close mine when we know its all working good
Zeglius
ZegliusOP•2mo ago
ah okay Oh k so the action actually runs
tulip🌷
tulip🌷•2mo ago
seems to get skipped tho
Zeglius
ZegliusOP•2mo ago
oh wait nvm mixed it with the current one Yeah probably because no package was changed also is missing the packit-run label Alright gonna try with a dummy commit on a package and see if triggers
tulip🌷
tulip🌷•2mo ago
i gotta just rebase mine
Zeglius
ZegliusOP•2mo ago
ok done, try rebasing
tulip🌷
tulip🌷•2mo ago
goig to my pc
Zeglius
ZegliusOP•2mo ago
nvm is working in my pr
Zeglius
ZegliusOP•2mo ago
No description
Zeglius
ZegliusOP•2mo ago
hell yeah wait there was no comment AH no its yours
tulip🌷
tulip🌷•2mo ago
yea i need to um rebase also remove the frankenstein thing
Zeglius
ZegliusOP•2mo ago
yeah i need that break smh
tulip🌷
tulip🌷•2mo ago
kill it with fire now that we dont need to use it no more OH SHIT IT WORKED LES GOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO @Zeglius the workflow but the comment failed....
tulip🌷
tulip🌷•2mo ago
No description
tulip🌷
tulip🌷•2mo ago
i think we need a write perm on something else content: write?
Zeglius
ZegliusOP•2mo ago
Okay so good, it triggers with comments made with an action But bad because now we need a separate account linked to fedora auth thingy
tulip🌷
tulip🌷•2mo ago
you sure? "account has no write access" what does that mean oh shit LOL also the "get all changed packages" thiung broke
Zeglius
ZegliusOP•2mo ago
regardless of that, there is a worse issue the bot doesnt interpret --package flag in gh comments
tulip🌷
tulip🌷•2mo ago
i think we want to just use the packit-cli thing
Zeglius
ZegliusOP•2mo ago
Ok but does it trigger the check runs in github is the question first we need to try doing it locally and see if it shows up
tulip🌷
tulip🌷•2mo ago
i mean, it'll show as an action workflow doing that right now locally
Zeglius
ZegliusOP•2mo ago
where?
tulip🌷
tulip🌷•2mo ago
just like any other action thing there wont be that cool "packit package thingmie"
Zeglius
ZegliusOP•2mo ago
No I mean where did show up for you?
tulip🌷
tulip🌷•2mo ago
No description
tulip🌷
tulip🌷•2mo ago
lioke this
Zeglius
ZegliusOP•2mo ago
so you still end up using github runners
tulip🌷
tulip🌷•2mo ago
ya its fine by me tbh it wont build locally
Zeglius
ZegliusOP•2mo ago
On top of that now you need to store secrets for the api token
tulip🌷
tulip🌷•2mo ago
i mean we cant... not use that
Zeglius
ZegliusOP•2mo ago
Also we still are vulnerable to secret leaks that way
tulip🌷
tulip🌷•2mo ago
shrug
Zeglius
ZegliusOP•2mo ago
At that point why even using packit
tulip🌷
tulip🌷•2mo ago
for the copr stuff like i dont know how we'll get over this shit i think we should use the frankenstein and have a separate bot account on our copr for triggering new builds OH @Zeglius we can just use copr cli with an account that has just enough privileges to build the packages we need the secret should always be replaceable anyways so even if we leak it it should be fine
Zeglius
ZegliusOP•2mo ago
you would still want to use packit for the ephemeral coprs
tulip🌷
tulip🌷•2mo ago
we can create a testing copr for each one on the workflow copr create packages_test then put all the builds there on like ublue-os-bot or something
Zeglius
ZegliusOP•2mo ago
No I refuse to reinvent the wheel
tulip🌷
tulip🌷•2mo ago
blud we cant use the wheel the wheel is just square shaped for us
Zeglius
ZegliusOP•2mo ago
Just replace the frankeinstein to use the _list_changed_packages recipe instead and done
tulip🌷
tulip🌷•2mo ago
we need smooth round wheels
Zeglius
ZegliusOP•2mo ago
Gonna ask in in the matrix channel if we can somewhat trigger individual builds other than by comments
tulip🌷
tulip🌷•2mo ago
aight! lwts see if we can get it going worst case we'll just improve on our own shit
Zeglius
ZegliusOP•2mo ago
@tulip🌷 btw if we can make the pr checks to only see if the matching package is building we are good then We trigger all the builds? Well, not my issue, should have they added filters for that alongside the mono repo support

Did you find this page helpful?