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

add nightly build and push #2297

Merged
merged 8 commits into from
Nov 18, 2023

Conversation

tuunit
Copy link
Member

@tuunit tuunit commented Oct 28, 2023

Description

Add nightly docker image build and publish workflow for the oauth2-proxy master branch. To always have a "bleeding edge" build of the latest merges for the community to test.

This PR requires the addition of the repository secrets for a service account with access to the quay.io repo:

  • REGISTRY_USERNAME_NIGHTLY
  • REGISTYR_PASSWORD_NIGHTLY

Motivation and Context

As the release cycle is rather slow some features might only be available on the master branch for weeks or month at a time. Therefore it would be great to have nightly builds of the master branch available.

How Has This Been Tested?

Build and publish workflow has been executed locally and on a forked repo.

Further notes

I took the liberty to update the doc images as part of this PR.

Checklist:

  • My change requires a change to the documentation or CHANGELOG.
  • I have updated the documentation/CHANGELOG accordingly.
  • I have created a feature (non-master) branch for my PR.

@tuunit tuunit changed the title [WIP] add nightly build and push add nightly build and push Oct 28, 2023
@tuunit tuunit marked this pull request as ready for review October 29, 2023 11:21
@tuunit tuunit requested a review from a team as a code owner October 29, 2023 11:21
Copy link
Member

@kvanzuijlen kvanzuijlen left a comment

Choose a reason for hiding this comment

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

quay.io registry needs to be updated, other than that I think its LGTM

@tuunit
Copy link
Member Author

tuunit commented Oct 31, 2023

quay.io registry needs to be updated, other than that I think its LGTM

@JoelSpeed let me know when you have created the new registry repo and I'll update it accordingly.

.github/workflows/nightly.yml Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@JoelSpeed
Copy link
Member

Are we creating a new repository or just using a nightly tag on the current repository? What do we want to create on Quay?

@tuunit
Copy link
Member Author

tuunit commented Nov 4, 2023

Are we creating a new repository or just using a nightly tag on the current repository? What do we want to create on Quay?

I think I would prefer to push the nightly builds to a separate repository. Otherwise it will just clutter the main repository.

In the case of a separate repository, I'll be renaming the necessary secrets to REGISTRY_USERNAME_NIGHTLY and REGISTYR_PASSWORD_NIGHTLY

@JoelSpeed
Copy link
Member

Ok, I'll set up a repo quay.io/oauth2-proxy/oauth2-proxy-nightly and we can use that one for the nightlies, will then set up the env vars as you've suggested

@tuunit
Copy link
Member Author

tuunit commented Nov 5, 2023

@JoelSpeed I updated the branch and the Makefile to point to the correct repository.

JoelSpeed
JoelSpeed previously approved these changes Nov 18, 2023
@JoelSpeed JoelSpeed merged commit bee7879 into oauth2-proxy:master Nov 18, 2023
8 checks passed
tuunit added a commit to tuunit/oauth2-proxy that referenced this pull request Nov 20, 2023
* add nightly build and push

* add date based nightly build tags

* only keep single multiarch image build and push

* add changelog

* add images to internal docs static files

* add docu for nightly builds

* remove unnecessary spaces

* update nightly repository
tuunit added a commit to tuunit/oauth2-proxy that referenced this pull request Nov 20, 2023
* add nightly build and push

* add date based nightly build tags

* only keep single multiarch image build and push

* add changelog

* add images to internal docs static files

* add docu for nightly builds

* remove unnecessary spaces

* update nightly repository
@ianroberts
Copy link
Contributor

Build and publish workflow has been executed locally and on a forked repo.

Now this is merged to master it might be a good idea to disable the nightly workflow from running in forks, since (a) I doubt you want forked repos trying to push to your quay.io repo and (b) the workflow runs will probably all fail anyway as the forks won't have access to the credential secrets.

I've suggested the obvious fix in #2330

tuunit added a commit to tuunit/oauth2-proxy that referenced this pull request Dec 17, 2023
* add nightly build and push

* add date based nightly build tags

* only keep single multiarch image build and push

* add changelog

* add images to internal docs static files

* add docu for nightly builds

* remove unnecessary spaces

* update nightly repository
tuunit added a commit to tuunit/oauth2-proxy that referenced this pull request Dec 18, 2023
* add nightly build and push

* add date based nightly build tags

* only keep single multiarch image build and push

* add changelog

* add images to internal docs static files

* add docu for nightly builds

* remove unnecessary spaces

* update nightly repository
JoelSpeed added a commit that referenced this pull request Dec 18, 2023
* Add GitHub groups (orgs/teams) support

* align code of getTeams with getOrgs to support Github Enterprise Server instances with different domain

* add documentation

* add missing import after rebase

* add nightly build and push (#2297)

* add nightly build and push

* add date based nightly build tags

* only keep single multiarch image build and push

* add changelog

* add images to internal docs static files

* add docu for nightly builds

* remove unnecessary spaces

* update nightly repository

* Issue 978: Fix Custom cookie name breaks redis for session (#1949)

* Issue 978: Fix Custom cookie name breaks redis for session (see #978)

* Issue 978: Fix Custom cookie name breaks redis for session (see #978)

* Update CHANGELOG.md

* Issue 978: Fix Custom cookie name breaks redis for session

* Issue 978: Fix Custom cookie name breaks redis for session

* Issue 978: Fix Custom cookie name breaks redis for session

* Issue 978: Fix Custom cookie name breaks redis for session

* Issue 978: Fix Custom cookie name breaks redis for session

* Issue 978: Fix Custom cookie name breaks redis for session

* Update CHANGELOG.md

---------

Co-authored-by: Nuno Borges <Nuno.Borges@ctw.bmwgroup.com>
Co-authored-by: Joel Speed <Joel.speed@hotmail.co.uk>

* Support http.AllowQuerySemicolons (#2248)

* Support http.AllowQuerySemicolons

* Docs

* Make it clear we are overriding the handler

* Update documentation for allow-query-semicolons

* Fix changelog format

* Fix formatting

---------

Co-authored-by: MickMake <github@mickmake.com>

* Add GitHub groups (orgs/teams) support

* align code of getTeams with getOrgs to support Github Enterprise Server instances with different domain

* add documentation

* fix changelog & documentation

* fix missing import

---------

Co-authored-by: Tobias Mayer <github@tobiasm.de>
Co-authored-by: Nuno Miguel Micaelo Borges <miguelborges99@gmail.com>
Co-authored-by: Nuno Borges <Nuno.Borges@ctw.bmwgroup.com>
Co-authored-by: Joel Speed <Joel.speed@hotmail.co.uk>
Co-authored-by: Tim White <tim.white@su.org.au>
Co-authored-by: MickMake <github@mickmake.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants