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

Adjustments for app-sre deployment #256

Merged
merged 6 commits into from
Dec 7, 2021
Merged

Adjustments for app-sre deployment #256

merged 6 commits into from
Dec 7, 2021

Conversation

bsquizz
Copy link
Contributor

@bsquizz bsquizz commented Nov 30, 2021

No description provided.

deploymentConfig: ibutsu-backend
spec:
containers:
- env:
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm happy to do this as a follow-up, but there is one more secret env var we need to run Ibutsu + Auth:
#252

@bsquizz bsquizz changed the title WIP: Adjustments for app-sre deployment Adjustments for app-sre deployment Dec 7, 2021
@bsquizz
Copy link
Contributor Author

bsquizz commented Dec 7, 2021

@john-dupuy @rsnyman you guys ok w/ merging this? I think the main impact would be just making sure that the value of the CELERY_BROKER_URL still gets read in and used for redis as expected on the existing PSI deployment

@john-dupuy
Copy link
Contributor

Yup, I think we're good to merge this in.

@john-dupuy john-dupuy merged commit db57b5f into master Dec 7, 2021
rsnyman added a commit to rsnyman/ibutsu-server that referenced this pull request Jan 18, 2022
* Small improvements to the test history tab
* Hide username/password related components when user login is disabled (ibutsu#279)
* Add test history tab to Result page (ibutsu#276)
* Address some issues raised by static analysis (ibutsu#278)
* Convert USER_LOGIN_ENABLED to bool if given as env var
* Get a user's primary e-mail on GitHub if they have no public e-mail address (ibutsu#272)
* Add a way to disabled basic auth for non-superadmins
* Add get_user_list endpoint
* Fix keycloak login (ibutsu#270)
* Update docker image names (ibutsu#267)
* Show login progress feedback (ibutsu#266)
* Add build_deploy script for App-SRE builds (ibutsu#265)
* A single jUnit XML file is a single test run, refactor the importer to take this into account
* Some code refactoring after static analysis. (ibutsu#245)
* Implement MetaFilter with separate field and value (ibutsu#225)
* Adjustments for app-sre deployment (ibutsu#256)
* Fix small FE bug on user profile page
* Add superadmin user after upgrading db
* Add a task for adding users/project owners
* Add superadmin to the ocp template files
* Add ability to create superadmin user on startup
* Allow logging in by hitting 'Enter'
* Require superadmin token for running admin task
* Update pods script to create an admin user, a project, standardise on echo, and make the output prettier. (ibutsu#247)
* Some small fixes to ENV vars
* Split templates into one file for each
* Update @greatsumini/react-facebook-login
* Fix Jenkins Job View
* Allow superadmins to update projects
* Support adding users to projects
* Add project info to the profile page
* Filter runs/results on user projects, if none specified
* Bump url-parse from 1.5.1 to 1.5.3 in /frontend (ibutsu#234)
* Bump tmpl from 1.0.4 to 1.0.5 in /frontend (ibutsu#233)
* Bump tar from 6.1.0 to 6.1.11 in /frontend (ibutsu#232)
* Add authentication and authorisation to Ibutsu
* Promote user property data to metadata
* Add babel core dependency
* Switch from 'babel-eslint' to '@babel/eslint-parser'
* Update and re-apply pre-commit (ibutsu#216)
* Support adding artifacts to test runs (ibutsu#215)
@rsnyman rsnyman mentioned this pull request Jan 18, 2022
jjaquish pushed a commit to jjaquish/ibutsu-server that referenced this pull request Aug 9, 2022
* Rework templates for app-sre deployment

* Updating templates

* Use f-strings

* Change name of 'flower' image

* Combine all backend components into 1 backend template

* Move celery broker config out of pgsql 'if' statement
jjaquish pushed a commit to jjaquish/ibutsu-server that referenced this pull request Aug 9, 2022
* Small improvements to the test history tab
* Hide username/password related components when user login is disabled (ibutsu#279)
* Add test history tab to Result page (ibutsu#276)
* Address some issues raised by static analysis (ibutsu#278)
* Convert USER_LOGIN_ENABLED to bool if given as env var
* Get a user's primary e-mail on GitHub if they have no public e-mail address (ibutsu#272)
* Add a way to disabled basic auth for non-superadmins
* Add get_user_list endpoint
* Fix keycloak login (ibutsu#270)
* Update docker image names (ibutsu#267)
* Show login progress feedback (ibutsu#266)
* Add build_deploy script for App-SRE builds (ibutsu#265)
* A single jUnit XML file is a single test run, refactor the importer to take this into account
* Some code refactoring after static analysis. (ibutsu#245)
* Implement MetaFilter with separate field and value (ibutsu#225)
* Adjustments for app-sre deployment (ibutsu#256)
* Fix small FE bug on user profile page
* Add superadmin user after upgrading db
* Add a task for adding users/project owners
* Add superadmin to the ocp template files
* Add ability to create superadmin user on startup
* Allow logging in by hitting 'Enter'
* Require superadmin token for running admin task
* Update pods script to create an admin user, a project, standardise on echo, and make the output prettier. (ibutsu#247)
* Some small fixes to ENV vars
* Split templates into one file for each
* Update @greatsumini/react-facebook-login
* Fix Jenkins Job View
* Allow superadmins to update projects
* Support adding users to projects
* Add project info to the profile page
* Filter runs/results on user projects, if none specified
* Bump url-parse from 1.5.1 to 1.5.3 in /frontend (ibutsu#234)
* Bump tmpl from 1.0.4 to 1.0.5 in /frontend (ibutsu#233)
* Bump tar from 6.1.0 to 6.1.11 in /frontend (ibutsu#232)
* Add authentication and authorisation to Ibutsu
* Promote user property data to metadata
* Add babel core dependency
* Switch from 'babel-eslint' to '@babel/eslint-parser'
* Update and re-apply pre-commit (ibutsu#216)
* Support adding artifacts to test runs (ibutsu#215)
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.

None yet

2 participants