-
Couldn't load subscription status.
- Fork 81
Splitting CI images into separate ones #712
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
Conversation
1e70ec0 to
c99f80d
Compare
de2b031 to
c902aea
Compare
|
Progress! Env tests passing! |
974dc76 to
6da38f8
Compare
|
Posting here, but some infos are also related to nextcloud/docker-ci#535
(from https://github.com/CSCfi/shibboleth-idp-dockerized/blob/master/opt/shibboleth-idp/conf/credentials-oidc.xml, haven't compared, but should be the same I guess, note the order, the file in the container is called After that: 😎 |
|
@SystemKeeper muchos gracias! Some changes i had already locally like that, but you also saved me some more digging ❤️ |
- adds integration.yml for integration tests - …where 389ds is added as service container and initialized in a later step after checkout - …and where a shibboleth server is also added service container TODOS: - [ ] double check integration tests config/paths/etc (was copy paste) Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
024b0d7 to
91254f1
Compare
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
91254f1 to
27e75f9
Compare
|
will remove require on drone once it's merged – there are no tests for drone anymore and the file was removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good 👍
Long way but nice work 🥳
- garm runners often run into host system issues and render the test unreliable (Error response from daemon: symlink /proc/mounts/var/lib/docker/fuse-overlayfs/$hash-init/merged/etc/mtab: file exists) Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
|
/backport to stable-5.2 |
|
The backport to stable-5.2 failed. Please do this backport manually. # Switch to the target branch and update it
git checkout stable-5.2
git pull origin stable-5.2
# Create the new backport branch
git checkout -b fix/foo-stable-5.2
# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123
# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable-5.2More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport |


Resolves #688 and should make it easier to maintain
Far from ready, contains temporary code (e.g. concurring drone.yml and MSGH Action)
TODOs in the commit message while in draft