Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Run tests and build also for ELN repository #2868

Merged
merged 1 commit into from
Nov 24, 2020
Merged

Run tests and build also for ELN repository #2868

merged 1 commit into from
Nov 24, 2020

Conversation

jkonecny12
Copy link
Member

Easiest way how to test ELN the same way as we do with Rawhide.

@jkonecny12 jkonecny12 added the master Please, use the `f39` label instead. label Sep 24, 2020
poncovka
poncovka approved these changes Sep 24, 2020
View reviewed changes
Copy link
Contributor

@poncovka poncovka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@jkonecny12
Copy link
Member Author

/packit build

1 similar comment
@TomasTomecek
Copy link
Contributor

/packit build

@TomasTomecek
Copy link
Contributor

The build is now failing on wrong GPG keys for the latest ELN compose ^_^

@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

heh, let's try it once again ;)

@jkonecny12
Copy link
Member Author

@TomasTomecek don't you know why this failure happened?

VladimirSlavik
VladimirSlavik approved these changes Oct 8, 2020
View reviewed changes
Copy link
Contributor

@VladimirSlavik VladimirSlavik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you!

@TomasTomecek
Copy link
Contributor

@TomasTomecek don't you know why this failure happened?

damn¸ we used to have these kind of problems in the past but it should have been fixed ~3 weeks ago - apparently not :/

@TomasTomecek
Copy link
Contributor

/packit build

@TomasTomecek
Copy link
Contributor

this time it fails to compile :)

Traceback (most recent call last):
  File "/usr/bin/g-ir-scanner", line 99, in <module>
    sys.exit(scanner_main(sys.argv))
  File "/usr/lib64/gobject-introspection/giscanner/scannermain.py", line 606, in scanner_main
    main.transform()
  File "/usr/lib64/gobject-introspection/giscanner/maintransformer.py", line 74, in transform
    self._add_standalone_doc_sections()
  File "/usr/lib64/gobject-introspection/giscanner/maintransformer.py", line 114, in _add_standalone_doc_sections
    self._namespace.append(node)
  File "/usr/lib64/gobject-introspection/giscanner/ast.py", line 461, in append
    raise ValueError("Namespace conflict: %r" % (node, ))
ValueError: Namespace conflict: DocSection('BaseWindow')
make[4]: *** [/usr/share/gobject-introspection-1.0/Makefile.introspection:156: AnacondaWidgets-3.3.gir] Error 1
make[4]: Leaving directory '/builddir/build/BUILD/anaconda-34.5/widgets/src'
make[3]: Leaving directory '/builddir/build/BUILD/anaconda-34.5/widgets'
make[3]: *** [Makefile:451: all-recursive] Error 1
make[2]: *** [Makefile:383: all] Error 2
make[2]: Leaving directory '/builddir/build/BUILD/anaconda-34.5/widgets'
make[1]: *** [Makefile:547: all-recursive] Error 1
make: *** [Makefile:441: all] Error 2
make[1]: Leaving directory '/builddir/build/BUILD/anaconda-34.5'

@martinpitt
Copy link
Contributor

To fix the compilation issue, you need to rebase to current master, to pick up commit 6dab27a

@jkonecny12
Copy link
Member Author

Rebased on top of master.

@jkonecny12 jkonecny12 mentioned this pull request Oct 12, 2020
Merged
@jkonecny12
Copy link
Member Author

/packit build

@TomasTomecek
Copy link
Contributor

Hm, ELN doesn't seem to be super-stable (last time GPG keys were wrong): Module or Group 'buildsys-build' is not available., same thing happening for me locally.

@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

So the ELN build should work now, however, ELN is broken because of the missing buildsys-build group.

@jkonecny12
Copy link
Member Author

Rebased

@jkonecny12
Copy link
Member Author

/packit build

@TomasTomecek
Copy link
Contributor

Module or Group 'buildsys-build' is not available. still the same :/

@jkonecny12
Copy link
Member Author

Fix is already in progress rpm-software-management/mock#649

@jkonecny12
Copy link
Member Author

/packit build

1 similar comment
@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

Still not there. Hope the change will take place soon.

@jkonecny12
Copy link
Member Author

jkonecny12 commented Oct 29, 2020 •
edited
Loading

/packit build

@jkonecny12
Copy link
Member Author

/packit build

2 similar comments
@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

jkonecny12 commented Nov 11, 2020 •
edited
Loading

Finally we are unblocked by the COPR and mock. However, we have to solve these issues:

Wrote: /builddir/build/SRPMS/anaconda-34.9-1.20201111161211144360.pr2868.18.gb060aa563.el105.src.rpm
No matches found for the following disable plugin patterns: local, spacewalk
Copr repository                                 187 kB/s | 3.3 kB     00:00    
Fedora - ELN BaseOS - Developmental packages fo 111 kB/s | 2.3 kB     00:00    
Fedora - ELN AppStream - Developmental packages 131 kB/s | 2.8 kB     00:00    
No matching package to install: 'glade-devel'
No matching package to install: 'glib2-doc'
No matching package to install: 'gobject-introspection-devel'
No matching package to install: 'gtk-doc'
No matching package to install: 'gtk3-devel-docs >= 3.22.17'
No matching package to install: 'libarchive-devel >= 3.0.4'
No matching package to install: 'libgnomekbd-devel'
No matching package to install: 'libtimezonemap-devel >= 0.4.1-2'
No matching package to install: 'libxklavier-devel >= 5.4'
No matching package to install: 'python3-nose'
Not all dependencies satisfied
Error: Some packages could not be found.

@jkonecny12 jkonecny12 mentioned this pull request Nov 11, 2020
Merged
@jkonecny12
Copy link
Member Author

Blocked by rpm-software-management/mock#658

@jkonecny12 jkonecny12 added the blocked Don't merge this pull request! label Nov 11, 2020
@martinpitt
Copy link
Contributor

@jkonecny12 : I ran into the missing deps problem in #2987 as well. My first container only had ELN sources, but then I kept the rawhide sources to get the missing ones. I haven't found the equivalent of a "buildroot" compose, and even the fedora-eln-repos package requires fedora-rawhide-repos -- so it all yells "use rawhide"..

@jkonecny12
Copy link
Member Author

/packit build

1 similar comment
@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

So we're slowly moving. Adding the CRB missing repository to mock helped a lot, however, it's still not done. We're still missing these two packages for some reason:

  • gtk3-devel-docs
  • libtimezonemap-devel

and also:

  • python3-nose

but that is kind of expected.

@jkonecny12
Copy link
Member Author

This should be the last piece: rpm-software-management/mock#659

@jkonecny12
Copy link
Member Author

/packit build

@jkonecny12
Copy link
Member Author

jkonecny12 commented Nov 23, 2020 •
edited
Loading

Hello @thrix ,

I'm getting

[12:10:07] [E] Pipeline reported an exception in module 'cruncher': Could not match string 'fedora-eln-x86_64' with any pattern

https://console-testing-farm.apps.ci.centos.org/pipeline/2dee9368-36b5-43b0-a83c-adbf385219f3

from Cruncher. Is that expected or will you enable also ELN testing?

@jkonecny12
Build Anaconda for ELN
55fb14f 
We would like to also run tests on ELN in the future but right now it seems that
Cruncher does not support that.
@jkonecny12
Copy link
Member Author

Changed configuration to only build in COPR for ELN because Cruncher does not support ELN right now.

@packit-as-a-service
Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/rhinstaller-anaconda-2868
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@jkonecny12
Copy link
Member Author

Finally we have a working ELN builds on Packit!

Thanks everyone for great cooperation!

@jkonecny12 jkonecny12 mentioned this pull request Nov 23, 2020
Merged
martinpitt
martinpitt suggested changes Nov 23, 2020
View reviewed changes
.packit.yml Show resolved Hide resolved
@VladimirSlavik
Copy link
Contributor

Thank you!

On the other hand, given the current success rate of packit runs, I wonder how much this really helps.

@martinpitt martinpitt mentioned this pull request Nov 23, 2020
Merged
@jkonecny12
Copy link
Member Author

Thank you!

On the other hand, given the current success rate of packit runs, I wonder how much this really helps.

The new ELN should always work. It does not use testing farm cluster which is failing right now. This is just between Packit and COPR and both these are more or less stable.

@jkonecny12 jkonecny12 removed the blocked Don't merge this pull request! label Nov 24, 2020
@jkonecny12 jkonecny12 merged commit 8c34cf1 into rhinstaller:master Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VladimirSlavik VladimirSlavik VladimirSlavik approved these changes

@poncovka poncovka poncovka approved these changes

@martinpitt martinpitt martinpitt approved these changes

Assignees
No one assigned
Labels
master Please, use the `f39` label instead.
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jkonecny12 @TomasTomecek @martinpitt @VladimirSlavik @poncovka