Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

Configure OBS instance to perform Clang builds #9

Merged
merged 11 commits into from Aug 22, 2018

Conversation

athos-ribeiro
Copy link
Contributor

This commit ports the salt states available at
https://github.com/athos-ribeiro/salt-obs to provision
irill8.siege.inria.fr with capabilities to monitor Debian
archives and trigger builds for newly accepted packages
substituting GCC for Clang binaries.

athos-ribeiro and others added 2 commits August 14, 2018 02:23
This commit ports the salt states available at
https://github.com/athos-ribeiro/salt-obs to provision
irill8.siege.inria.fr with capabilities to monitor Debian
archives and trigger builds for newly accepted packages
substituting GCC for Clang binaries.
README.md Outdated
@@ -0,0 +1,17 @@
# Open Build Service at irill8.siege.inria.fr
Copy link
Contributor

Choose a reason for hiding this comment

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

This salt instance is doing other things. Please add something on the top of this file!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

obs-server.sls Outdated
@@ -1,4 +1,21 @@
install obs server packages:
/etc/apt/sources.list:
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we want to do that. We should instead add the repo into
/etc/apt/sources.list.d/ instead

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK. I am looking into it

Copy link
Contributor

Choose a reason for hiding this comment

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

ping?

obs-server.sls Outdated
host.present:
- ip: 127.0.0.1

refresh_packages_db:
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't necessary, add a repo the salt way will do it
https://docs.saltstack.com/en/latest/ref/states/all/salt.states.pkgrepo.html

Copy link
Contributor

Choose a reason for hiding this comment

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

ping?

obs-server.sls Outdated
- obs-build
- fromrepo: stretch-backports

install_libsolv_from_testing:
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a comment (or link to your blog post) explaining why we are doing that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

obs-server.sls Outdated
- libsolvext0
- fromrepo: buster

/usr/share/obs/api/Gemfile:
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

if (pkg['dist'] in ['unstable', 'sid']) and 'source' in pkg['arch']:
trigger_build(pkg)
# Uncomment the following lines if you want to remove the email afterwards
# M.store(num, '+FLAGS', '\\Deleted')
Copy link
Contributor

Choose a reason for hiding this comment

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

why we are not doing that? (removing the email)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No specific reason. I Just uncommented the lines :)

<configuration>
<title>Debian LLVM OBS builder</title>
<description> &lt;p class="description"&gt;
The Debian LLVM OBS builder is used to both distribute upstream LLVM .deb packages
Copy link
Contributor

Choose a reason for hiding this comment

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

you can trim the description just to keep your part!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

<disable_publish_for_branches>on</disable_publish_for_branches>
<admin_email>unconfigured@openbuildservice.org</admin_email>
<schedulers>
<arch>armv7l</arch>
Copy link
Contributor

Choose a reason for hiding this comment

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

you are also remove arm devices

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -0,0 +1,7 @@
deb http://httpredir.debian.org/debian stretch main
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove this file

Copy link
Contributor

Choose a reason for hiding this comment

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

ping?

echo 'Preparing obs-service-clang-build package...'
mkdir obs-service-clang-build
cd obs-service-clang-build/
wget 'https://www.ime.usp.br/~athoscr/files/obs/obs-service-clang-build_0.1-1.debian.tar.xz'
Copy link
Contributor

Choose a reason for hiding this comment

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

please add this to this repo and use that. I would like to avoid external dependencies

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is it ok to add the tarballs here then?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, better than on thirdparty website.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@sylvestre sylvestre merged commit 30c7d48 into opencollab:master Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants