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

Automate building of packages in OBS #9

Closed
wants to merge 1 commit into from

Conversation

aazam-gh
Copy link
Contributor

@aazam-gh aazam-gh commented Aug 22, 2024

This PR is part of the Feilong Packaging Process to automate building of packages (zthin and zvmsdk) for the 3 distros (Ubuntu, RHEL, SLES) on OBS.

In order for this automation to work, we must use a webhook in order for OBS to receive events from the github action to trigger the rebuild of the packages with the latest source files.
In order to use the webhook, we require a Personal Access Token (PAT) which will be used by OBS to generate the url for the webhook.

The working of the action is as follows:

  • When build-zvmsdk receives the dispatched event from this (feilong) github action, it first runs a few script files to generate new tarballs from the latest version of the feilong master branch.
  • After creating all the source files, it downloads them as artifacts and pushes them to the master branch of build-zvmsdk.
  • Once build-zvmsdk has a push, it triggers the webhook with OBS which causes each package hosted on OBS to download the required tarballs and/or .dsc files followed by triggering a rebuild for each package.

This PR handles the development on feilong side.

Please create a PAT on github for this repository.
The webhook will then be created with the Push event.

@mfriesenegger
Copy link

mfriesenegger commented Aug 22, 2024

@aazam-gh @Bischoff Is best practice to squash all of the commits for a PR?

Comment on lines 1 to 9
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>404 Not Found</title>
</head><body>
<h1>Not Found</h1>
<p>The requested URL was not found on this server.</p>
<hr>
<address>Apache/2.4.52 (Ubuntu) Server at download.kde.org Port 443</address>
</body></html>

Choose a reason for hiding this comment

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

Should this file be removed from the PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

Choose a reason for hiding this comment

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

Yes. Unless you really meant to have different commits addressing distinct topics.

@aazam-gh
Copy link
Contributor Author

Closed in favour of #10 due to rebase conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants