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

Update OpenSSH downstream branch to OQS-v9 #1898

Merged
merged 5 commits into from
Aug 22, 2024
Merged

Conversation

SWilson4
Copy link
Member

As per open-quantum-safe/openssh#166.

* [ ] Does this PR change the input/output behaviour of a cryptographic algorithm (i.e., does it change known answer test values)? (If so, a version bump will be required from x.y.z to x.(y+1).0.)
* [ ] Does this PR change the list of algorithms available -- either adding, removing, or renaming? Does this PR otherwise change an API? (If so, PRs in fully supported downstream projects dependent on these, i.e., oqs-provider will also need to be ready for review and merge by the time this is merged.)

@SWilson4
Copy link
Member Author

@geedo0 You'll need to set the workflow in OpenSSH to be triggered on workflow_dispatch events.

@geedo0
Copy link

geedo0 commented Aug 21, 2024

Ack, added the trigger to the PR here.

@SWilson4
Copy link
Member Author

Apparently the GitHub trigger won't do anything unless the downstream workflow is on the default branch: see open-quantum-safe/openssh#168 (comment) for discussion.

Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
Signed-off-by: Spencer Wilson <[email protected]>
This reverts commit ba4a551.

Signed-off-by: Spencer Wilson <[email protected]>
@SWilson4 SWilson4 marked this pull request as ready for review August 21, 2024 18:35
@SWilson4 SWilson4 requested a review from dstebila as a code owner August 21, 2024 18:35
@SWilson4
Copy link
Member Author

Trigger tested on commit caea971: see the upstream and and downstream runs. For testing, I disabled the other downstream triggers and set the workflow to be triggered on pushes to this branch; the most recent commit reverts these changes.

@SWilson4 SWilson4 merged commit 66f713f into main Aug 22, 2024
46 of 48 checks passed
@SWilson4 SWilson4 deleted the sw-update-openssh-branch branch August 22, 2024 13:51
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.

4 participants