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.

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

Build: Error - Cannot package the path '/tmp/dist/summary/check-plugins/The Unlicense #733

Open
5 of 8 tasks
Domi-cc opened this issue Dec 29, 2023 · 2 comments
Open
5 of 8 tasks
Labels
bug Something isn't working
Milestone

Comments

@Domi-cc
Copy link

Domi-cc commented Dec 29, 2023

This issue respects the following points:

Which variant of the Monitoring Plugins do you use?

  • .rpm/.deb package from repo.linuxfabrik.ch
  • Compiled for Linux (.tar/.zip from download.linuxfabrik.ch)
  • Compiled for Windows (from download.linuxfabrik.ch)
  • Source Code from GitHub

Bug description

i got an error while building the Plugins with the included buildscripts:

{:timestamp=>"2023-12-29T11:46:57.120435+0000", :message=>"Invalid package configuration: Cannot package the path '/tmp/dist/summary/check-plugins/The Unlicense', does it exist?", :level=>:error}

I used a debian12 docker container to build the packages:

  docker run \
  --rm \
  --pull always \
  --mount type=bind,source="/dockerdata/linuxfabrik_plugin_builder/repo-linuxfabrik-monitoring-plugins",target=/repos/monitoring-plugins,readonly \
  --mount type=bind,source="/mnt/shareall/linuxfabrik-monitoring-plugins/tmp",target=/build \
  debian:12 \
  bash /repos/monitoring-plugins/build/debian12/build.sh

the first mount is the git repo from master, the second mount is only to output the packages from the container.

Steps to reproduce - Plugin call

--

Steps to reproduce - Data

No response

Environment

debian12 docker container

Plugin Version

--

Python version

No response

List of Python modules

No response

Additional Information

No response

@Domi-cc Domi-cc added the bug Something isn't working label Dec 29, 2023
@Domi-cc
Copy link
Author

Domi-cc commented Dec 29, 2023

it seems, the problem occurs, because PACKAGE_VERSION and PACKAGE_ITERATION is not set.

So it would be great, to have default values there or change the script to raise an error if parameters are not set.

@Domi-cc
Copy link
Author

Domi-cc commented Dec 29, 2023

next trap: please also add to the script a check, if the lib folder is there ;)

@markuslf markuslf changed the title Build Error: Cannot package the path '/tmp/dist/summary/check-plugins/The Unlicense Build: Error - Cannot package the path '/tmp/dist/summary/check-plugins/The Unlicense Dec 9, 2024
@markuslf markuslf added this to the M006 milestone Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants