Add Packit Integration by justin-stephenson · Pull Request #274 · Scribery/tlog

justin-stephenson · 2019-12-05T14:41:26Z

Add basic setup for https://packit.dev/ triggering RPM builds in pull requests, tests to be added later.

coveralls · 2019-12-05T14:55:06Z

Coverage remained the same at 29.351% when pulling 5f4fe43 on justin-stephenson:packit_service_integrate into 7c18616 on Scribery:master.

TomasTomecek · 2019-12-10T11:18:43Z

/packit copr-build

packit-as-a-service · 2019-12-10T11:25:34Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

make: *** No rule to make target 'dist'.  Stop.

Return code: 2

packit-as-a-service · 2019-12-10T14:48:19Z

There was a problem in the environment the service is running in:

(403)
Reason: Forbidden
HTTP response headers: HTTPHeaderDict({'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Tue, 10 Dec 2019 14:48:18 GMT', 'Content-Length': '425'})
HTTP response body:
{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "pods \"docker-io-usercont-sandcastle-20191210-144818413794\" is forbidden: exceeded quota: 74746f6d6563656b407265646861742e636f6d-timebound, requested: limits.memory=512Mi, used: limits.memory=2Gi, limited: limits.memory=2Gi",
  "reason": "Forbidden",
  "details": {
    "name": "docker-io-usercont-sandcastle-20191210-144818413794",
    "kind": "pods"
  },
  "code": 403
}

You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Please, contact @packit-service/the-packit-team if the re-trigger did not help.

packit-as-a-service · 2019-12-10T16:33:31Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

configure: loading site script /usr/share/config.site
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking whether make supports the include directive... yes (GNU style)
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking whether gcc understands -c and -o together... (cached) yes
checking dependency style of gcc... (cached) gcc3
checking for gcc option to accept ISO C99... none needed
checking for gcc option to accept ISO Standard C... (cached) none needed
checking for a sed that does not truncate output... /usr/bin/sed
checking for ar... ar
checking the archiver (ar) interface... ar
checking for m4... /usr/bin/m4
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking how to print strings... printf
checking for a sed that does not truncate output... (cached) /usr/bin/sed
checking for fgrep... /usr/bin/grep -F
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /usr/bin/nm -B
checking the name lister (/usr/bin/nm -B) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-pc-linux-gnu file names to x86_64-pc-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for /usr/bin/ld option to reload object files... -r
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for mt... no
checking if : is a manifest tool... no
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... no
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking whether gcc is Clang... no
checking whether pthreads work with -pthread... yes
checking for joinable pthread attribute... PTHREAD_CREATE_JOINABLE
checking whether more special flags are required for pthreads... no
checking for PTHREAD_PRIO_INHERIT... yes
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for JSON... no
configure: error: Package requirements (json-c) were not met:

Package 'json-c', required by 'virtual:world', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables JSON_CFLAGS
and JSON_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

Return code: 1

packit-as-a-service · 2019-12-10T16:48:35Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

Error: This command has to be run under the root user.

Return code: 1

justin-stephenson · 2019-12-11T14:07:13Z

/packit copr-build

packit-as-a-service · 2019-12-11T14:10:46Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

Error: This command has to be run under the root user.

Return code: 1

packit-as-a-service · 2019-12-11T14:24:50Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

configure: loading site script /usr/share/config.site
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking whether make supports the include directive... yes (GNU style)
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking whether gcc understands -c and -o together... (cached) yes
checking dependency style of gcc... (cached) gcc3
checking for gcc option to accept ISO C99... none needed
checking for gcc option to accept ISO Standard C... (cached) none needed
checking for a sed that does not truncate output... /usr/bin/sed
checking for ar... ar
checking the archiver (ar) interface... ar
checking for m4... /usr/bin/m4
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking how to print strings... printf
checking for a sed that does not truncate output... (cached) /usr/bin/sed
checking for fgrep... /usr/bin/grep -F
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /usr/bin/nm -B
checking the name lister (/usr/bin/nm -B) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-pc-linux-gnu file names to x86_64-pc-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for /usr/bin/ld option to reload object files... -r
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for mt... no
checking if : is a manifest tool... no
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... no
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking whether gcc is Clang... no
checking whether pthreads work with -pthread... yes
checking for joinable pthread attribute... PTHREAD_CREATE_JOINABLE
checking whether more special flags are required for pthreads... no
checking for PTHREAD_PRIO_INHERIT... yes
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for JSON... no
configure: error: Package requirements (json-c) were not met:

Package 'json-c', required by 'virtual:world', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables JSON_CFLAGS
and JSON_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

Return code: 1

TomasTomecek · 2019-12-12T13:07:13Z

/packit build

TomasTomecek · 2019-12-12T13:07:34Z

new sandbox image with json-c should be up, let's see

packit-as-a-service · 2019-12-12T13:14:32Z

There was an error while creating a SRPM. You can re-trigger copr build by adding a comment (/packit copr-build) into this pull request.

Output:

configure: loading site script /usr/share/config.site
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking whether make supports the include directive... yes (GNU style)
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking whether gcc understands -c and -o together... (cached) yes
checking dependency style of gcc... (cached) gcc3
checking for gcc option to accept ISO C99... none needed
checking for gcc option to accept ISO Standard C... (cached) none needed
checking for a sed that does not truncate output... /usr/bin/sed
checking for ar... ar
checking the archiver (ar) interface... ar
checking for m4... /usr/bin/m4
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking how to print strings... printf
checking for a sed that does not truncate output... (cached) /usr/bin/sed
checking for fgrep... /usr/bin/grep -F
checking for ld used by gcc... /usr/bin/ld
checking if the linker (/usr/bin/ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... /usr/bin/nm -B
checking the name lister (/usr/bin/nm -B) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-pc-linux-gnu file names to x86_64-pc-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for /usr/bin/ld option to reload object files... -r
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse /usr/bin/nm -B output from gcc object... ok
checking for sysroot... no
checking for a working dd... /usr/bin/dd
checking how to truncate binary pipes... /usr/bin/dd bs=4096 count=1
checking for mt... no
checking if : is a manifest tool... no
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... no
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking whether gcc is Clang... no
checking whether pthreads work with -pthread... yes
checking for joinable pthread attribute... PTHREAD_CREATE_JOINABLE
checking whether more special flags are required for pthreads... no
checking for PTHREAD_PRIO_INHERIT... yes
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for JSON... no
configure: error: Package requirements (json-c) were not met:

Package 'json-c', required by 'virtual:world', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables JSON_CFLAGS
and JSON_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

Return code: 1

TomasTomecek · 2019-12-12T13:20:08Z

I wonder, should it be json-c or json-c-devel?

justin-stephenson · 2019-12-12T14:02:00Z

@TomasTomecek My mistake, it should be json-c-devel, systemd-devel, and libcurl-devel.

TomasTomecek · 2020-01-09T09:00:54Z

Finally got back to this after the winter holidays. I added the -devel packages to our sandbox - the changes should be live on Monday.

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>

packit: use actions to make the build work

packit-as-a-service · 2020-01-29T20:12:53Z

There was an error while creating SRPM. You can re-trigger build by adding a comment (/packit copr-build) into this pull request.

Output:

Preparing of the upstream to the SRPM build failed: Failed to download file from URL https://github.com/Scribery/tlog/releases/download/vv4/tlog-v4.tar.gz. Reason: 'Not Found'.

dhodovsk · 2020-02-03T14:35:24Z

The issue is:

create-archive creates file tlog-7.tar.gz
get-current-version returns v4
during fix-spec, version is set to v4 and in source the expected file is tlog-v4.tar.gz (from %{name}-%{version}.tar.gz) but there is only tlog-7.tar.gz

justin-stephenson · 2020-02-03T16:00:44Z

@dhodovsk Hello, in the tlog master github repo should return '7'

[jstephen@agalloch tlog]$ git describe --abbrev=0
7

And in the packet configuration we have:

  get-current-version:
  - git describe --abbrev=0

Could you help me understand if I am missing something here?

TomasTomecek · 2020-02-06T12:21:13Z

I actually have the same problem as Justin: it works for me locally with packit CLI, but in the service we have the problem which Dominika described.

dhodovsk · 2020-02-06T13:06:01Z

So I've tried it again and it seems the tag is only in https://github.com/justin-stephenson/tlog fork:

[root@30ac66c95f97 src]# git clone https://github.com/justin-stephenson/tlog
Cloning into 'tlog'...
remote: Enumerating objects: 50, done.
remote: Counting objects: 100% (50/50), done.
remote: Compressing objects: 100% (41/41), done.
remote: Total 4894 (delta 23), reused 17 (delta 9), pack-reused 4844
Receiving objects: 100% (4894/4894), 1.34 MiB | 4.04 MiB/s, done.
Resolving deltas: 100% (3561/3561), done.
[root@30ac66c95f97 src]# cd tlog/
[root@30ac66c95f97 tlog]# git describe --abbrev=0
7

But in Scribery:

[root@30ac66c95f97 src]# git clone https://github.com/Scribery/tlog
Cloning into 'tlog'...
remote: Enumerating objects: 25, done.
remote: Counting objects: 100% (25/25), done.
remote: Compressing objects: 100% (18/18), done.
remote: Total 4730 (delta 13), reused 10 (delta 7), pack-reused 4705
Receiving objects: 100% (4730/4730), 1.29 MiB | 2.77 MiB/s, done.
Resolving deltas: 100% (3448/3448), done.
[root@30ac66c95f97 src]# cd tlog/
[root@30ac66c95f97 tlog]# git describe --abbrev=0
v4

When packit checks out the PR to make changes, it only does the alternative of git fetch origin pull/274/head, which doesn't fetch the tags.

@TomasTomecek is this expected or should we do something about it in packit?

TomasTomecek · 2020-03-12T11:36:16Z

I'd say that we should definitely fetch tags in packit. Shall we open an issue for that?

justin-stephenson force-pushed the packit_service_integrate branch from ce227f7 to b8d6b82 Compare December 10, 2019 14:48

justin-stephenson force-pushed the packit_service_integrate branch from b8d6b82 to 6d92aa5 Compare December 10, 2019 16:27

justin-stephenson force-pushed the packit_service_integrate branch from 6d92aa5 to 18d6527 Compare December 10, 2019 16:45

Add Packit Integration

5e52448

justin-stephenson force-pushed the packit_service_integrate branch from 18d6527 to 5e52448 Compare December 11, 2019 14:10

TomasTomecek mentioned this pull request Jan 9, 2020

add -devel pkgs for tlog packit/sandcastle#44

Merged

TomasTomecek and others added 2 commits January 29, 2020 16:58

packit: use actions to make the build work

99aa695

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>

Merge pull request #6 from TomasTomecek/packit

5f4fe43

packit: use actions to make the build work

Comments

Conversation

justin-stephenson commented Dec 5, 2019

Uh oh!

coveralls commented Dec 5, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TomasTomecek commented Dec 10, 2019

Uh oh!

packit-as-a-service bot commented Dec 10, 2019

Uh oh!

packit-as-a-service bot commented Dec 10, 2019

Uh oh!

packit-as-a-service bot commented Dec 10, 2019

Uh oh!

packit-as-a-service bot commented Dec 10, 2019

Uh oh!

justin-stephenson commented Dec 11, 2019

Uh oh!

packit-as-a-service bot commented Dec 11, 2019

Uh oh!

packit-as-a-service bot commented Dec 11, 2019

Uh oh!

TomasTomecek commented Dec 12, 2019

Uh oh!

TomasTomecek commented Dec 12, 2019

Uh oh!

packit-as-a-service bot commented Dec 12, 2019

Uh oh!

TomasTomecek commented Dec 12, 2019

Uh oh!

justin-stephenson commented Dec 12, 2019

Uh oh!

TomasTomecek commented Jan 9, 2020

Uh oh!

packit-as-a-service bot commented Jan 29, 2020

Uh oh!

dhodovsk commented Feb 3, 2020

Uh oh!

justin-stephenson commented Feb 3, 2020

Uh oh!

TomasTomecek commented Feb 6, 2020

Uh oh!

dhodovsk commented Feb 6, 2020

Uh oh!

TomasTomecek commented Mar 12, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

coveralls commented Dec 5, 2019 •

edited

Loading