Update ci to f34 #4244
Update ci to f34 #4244
Conversation
AdamSaleh
force-pushed
the
update_ci_to_f34
branch
from
6c24486 to
dac1cae
Compare
devel/ci/cico.pipeline
Outdated
| @@ -161,16 +161,16 @@ node('bodhi') { | |||
|
|
|||
| try { | |||
| parallel( | |||
| f32: {test_release('f32')}, | |||
| f33: {test_release('f33')}, | |||
| f33: {test_release('f32')}, | |||
There was a problem hiding this comment.
Shouldn't it be
f33: {test_release('f33')}, f34: {test_release('f34')},
?
There was a problem hiding this comment.
Proposed changes:
LenkaSeg@93b432c
AdamSaleh
force-pushed
the
update_ci_to_f34
branch
2 times, most recently
from
0903aee to
d684412
Compare
devel/ci/Dockerfile-rawhide
Outdated
| @@ -1,7 +1,10 @@ | |||
| FROM registry.fedoraproject.org/fedora:rawhide | |||
| FROM registry.fedoraproject.org/fedora:35 | |||
There was a problem hiding this comment.
This does not seems correct, since rawhide is now f36. We should keep the rawhide label.
| @@ -1,4 +1,4 @@ | |||
| FROM bodhi-ci/f32 | |||
| FROM bodhi-ci/f34 | |||
There was a problem hiding this comment.
should probably rename the file too
| @@ -28,5 +24,3 @@ PyYAML | |||
| simplemediawiki | |||
| sqlalchemy | |||
| waitress | |||
| WebOb-GraphQL | |||
| whitenoise | |||
There was a problem hiding this comment.
I think whitenoise is used to serve the docs so we should keep it.
| jinja2 | ||
| markdown>=3.0 | ||
| prometheus_client |
There was a problem hiding this comment.
These 2 should probably still be there
| @@ -10,12 +10,8 @@ dogpile.cache | |||
| pyasn1-modules # Due to an unfortunate dash in its name, installs break if pyasn1 is installed first | |||
| fedora_messaging | |||
| feedgen>=0.7.0 | |||
| graphene | |||
| graphene-sqlalchemy | |||
| koji | |||
There was a problem hiding this comment.
Pretty sure we still want to have koji here :)
| @@ -1,4 +1,4 @@ | |||
| FROM bodhi-ci/f32 | |||
| FROM bodhi-ci/f34 | |||
There was a problem hiding this comment.
You will need to change
RUN python3 setup.py build && pip3 install --no-use-pep517 . && pip3 install koji
with
RUN python3 setup.py bdist_wheel && pip3 install ./dist/*.whl && pip3 install koji
to fix f34-integration-build
There was a problem hiding this comment.
Oh, also add python3-wheel to dnf install command, see 6f5d36d
There was a problem hiding this comment.
Done and done. f34-integration-build works now, thanks!
There was a problem hiding this comment.
You're welcome.
For the f32-* tests which are always required, I think there is a setting at project level in github that can only be changed by an admin... unfortunately, removing them from mergify config isn't enough.
There was a problem hiding this comment.
Found that setting as well. You think this looks good enough? I'd really like to merge this and then start looking at some of the stale PR's we have around the place, with the work of fixing and refactoring the integration tests to be postponed to a later date.
AdamSaleh
force-pushed
the
update_ci_to_f34
branch
2 times, most recently
from
1a99b70 to
0b0d353
Compare
Signed-off-by: Tomas Hrcka <thrcka@redhat.com>
Signed-off-by: Tomas Hrcka <thrcka@redhat.com>
Signed-off-by: Adam Saleh <asaleh@redhat.com>
Signed-off-by: Adam Saleh <asaleh@redhat.com>
…ds a new query allReleases that allows to fetch all Release objects from the database using GraphQL. Fixes fedora-infra#4024." This reverts commit 84fdcb4. Signed-off-by: Adam Saleh <asaleh@redhat.com>
This reverts commit fe46499. Signed-off-by: Adam Saleh <asaleh@redhat.com>
Signed-off-by: Adam Saleh <asaleh@redhat.com>
Fix test failure with unescaped sphinx version pin Disable repo that fails with rawhide integration build Remove rawhide from CI due to issue on jenkins builder Fixe mypy by installing additional types-* packages Signed-off-by: Adam Saleh <asaleh@redhat.com>
Signed-off-by: Tomas Hrcka <thrcka@redhat.com>
Signed-off-by: Adam Saleh <asaleh@redhat.com>
AdamSaleh
force-pushed
the
update_ci_to_f34
branch
from
0b0d353 to
7caee26
Compare
Update vagrant env and Dockerfiles to use f34 and make changes to CI and mergify