Skip to content

downgrade ubuntu in order to fix CI - #35580

Merged
blizzz merged 1 commit into
masterfrom
enh/noid/fix-CI
Dec 5, 2022
Merged

downgrade ubuntu in order to fix CI#35580
blizzz merged 1 commit into
masterfrom
enh/noid/fix-CI

Conversation

@szaimen

@szaimenszaimen commented Dec 3, 2022•
edited
Loading

Copy link
Copy Markdown
Contributor

Signed-off-by: Simon L szaimen@e.mail.de
image

@szaimen
downgrade ubuntu in order to fix CI
7964b86
Signed-off-by: Simon L <szaimen@e.mail.de>
@szaimenszaimen added the 2. developing Work in progress label Dec 3, 2022
@szaimenszaimen added this to the Nextcloud 26 milestone Dec 3, 2022
@szaimen
szaimen marked this pull request as ready for review December 3, 2022 22:35
@szaimenszaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Dec 3, 2022
@szaimen

szaimen commented Dec 3, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable25

@szaimen

szaimen commented Dec 3, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable24

@szaimen

szaimen commented Dec 3, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable23

@szaimen

szaimen commented Dec 3, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable22

@szaimen
szaimen requested a review from a teamDecember 4, 2022 09:22
@skjnldsv

skjnldsv commented Dec 4, 2022

Copy link
Copy Markdown
Member

WHat's the context of this? :)

@szaimen

szaimen commented Dec 4, 2022

Copy link
Copy Markdown
ContributorAuthor

WHat's the context of this? :)

See the failing phpunit tests e.g. in #35547 and in other PRs. For further context see the engineering chat.

@nickvergessen

nickvergessen commented Dec 4, 2022

Copy link
Copy Markdown
Member

Refering to an internal chat is not the best documentation. Just summarize it here?
Should also do it in the workflow template repository

@MichaIng

MichaIng commented Dec 4, 2022

Copy link
Copy Markdown
Member

I'd be also interested about the reason why it fails on Ubuntu Jammy runners. But good to know that there is a workaround for the CI failures 👍.

@szaimen

szaimen commented Dec 5, 2022

Copy link
Copy Markdown
ContributorAuthor

I updated the description 👍

@blizzz
blizzz merged commit 4196bf8 into masterDec 5, 2022
@blizzz
blizzz deleted the enh/noid/fix-CI branch December 5, 2022 09:59
@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable25 failed. Please do this backport manually.

@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable24 failed. Please do this backport manually.

@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable23 failed. Please do this backport manually.

@szaimen

szaimen commented Dec 5, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable25

@MichaIng

MichaIng commented Dec 5, 2022

Copy link
Copy Markdown
Member

Wouldn't it be better to apply the workaround in #32003 instead? However, doesn't matter so much, both can be a short-term workaround only. Looks like #25551 needs to be done not not force admins of recent distros, with server-side encryption enabled, to enable legacy OpenSSL ciphers.

@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable22 failed. Please do this backport manually.

@szaimen

szaimen commented Dec 5, 2022

Copy link
Copy Markdown
ContributorAuthor

/backport to stable25

This was referenced Dec 5, 2022
Merged
Merged
Merged
@szaimen

szaimen commented Dec 5, 2022

Copy link
Copy Markdown
ContributorAuthor

all backports done. Looks like the stable22 and below does not need this.

@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable25 failed. Please do this backport manually.

1 similar comment
@backportbot-nextcloud

backportbot-nextcloudBot commented Dec 5, 2022

Copy link
Copy Markdown

The backport to stable25 failed. Please do this backport manually.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessennickvergessennickvergessen approved these changes
@blizzzblizzzblizzz approved these changes
@kesselbkesselbAwaiting requested review from kesselb

Assignees

No one assigned

Labels

3. to reviewWaiting for reviews

Projects

None yet

Milestone

Nextcloud 26

Development

Successfully merging this pull request may close these issues.

5 participants

@szaimen@skjnldsv@nickvergessen@MichaIng@blizzz