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

Initial work for 2.5 #304

Merged
merged 4 commits into from
Jan 31, 2024
Merged

Conversation

jwendell
Copy link
Member

This is based on Dario's branch, plus one commit with a few adjustments to CI scripts and removal of some leftover files from rebase.

dcillera and others added 4 commits January 12, 2024 14:07
…ustRootOnly

The certificate verifification depth needed to change from 2 to 1 because
OpenSSL excludes the root AND leaf certs from the depth check, whereas
BoringSSL only excludes the leaf cert.

Signed-off-by: Ted Poole <tpoole@redhat.com>
Build is enabled in CI but tests are not.
@jwendell jwendell added the tide/merge-method-rebase Denotes a PR that should be rebased by tide when it merges. label Jan 30, 2024
@jwendell
Copy link
Member Author

  • These envoy prechecks will be go away after this PR is merged, I think. Last commit in this PR removes everything related to them.
  • Our CI script only builds Envoy for now. Tests are not running. We should re-enable them in a follow-up PR, when we will fix the failures.

@jwendell
Copy link
Member Author

@yxun the tests passed but the job failed at some post action. Can you please take a look?

@yxun
Copy link
Member

yxun commented Jan 30, 2024

@yxun the tests passed but the job failed at some post action. Can you please take a look?

What are that set of Github CI jobs ? It looks like in the ci directory. They are triggered by Github or bot

@yxun
Copy link
Member

yxun commented Jan 30, 2024

"Process did not finish before 10m0s timeout" we have some timeout limit in cleanup step. The namespace deletion was stuck. Actually, we don't need that step. The deprovision cluster step is enough for cleanup.

@yxun
Copy link
Member

yxun commented Jan 30, 2024

@jwendell
Copy link
Member Author

/retest

1 similar comment
@jwendell
Copy link
Member Author

/retest

@twghu twghu self-requested a review January 31, 2024 12:39
Copy link
Contributor

@twghu twghu left a comment

Choose a reason for hiding this comment

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

Ok as initial PR.

@openshift-merge-bot openshift-merge-bot bot merged commit 3181c70 into maistra:maistra-2.5 Jan 31, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
okay to merge size/XXL tide/merge-method-rebase Denotes a PR that should be rebased by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants