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.

Sign up for GitHub

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

Jump to bottom

Mpp prod move #541

Merged
merged 8 commits into from
Feb 6, 2024
Merged

Mpp prod move #541

merged 8 commits into from
Feb 6, 2024

Conversation

mfocko
Copy link
Member

@mfocko mfocko commented Nov 14, 2023 •
edited
Loading

Rebased on top of @majamassarini PR, so it's easy to retry next time.

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/9ce806c3a6414316b759f6b70a629f78

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 44s

jpopelka
jpopelka reviewed Nov 14, 2023
View reviewed changes
openshift/nginx.yml.j2 Outdated Show resolved Hide resolved
openshift/pushgateway.yml.j2 Outdated Show resolved Hide resolved
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/570b5bfe88cd433e96ed8f1291595e9c

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 44s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/20c115cbd7f941168c6faf5d949d5ed8

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 17s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/f9444b9f5a6e408abf863e53eaa3bbd7

✔️ pre-commit SUCCESS in 1m 42s
❌ deployment-tests RETRY_LIMIT in 7m 27s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/3c682b856ba643a1b513a739dfdf6120

✔️ pre-commit SUCCESS in 1m 39s
❌ deployment-tests RETRY_LIMIT in 7m 45s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/6b4bab32e821425e8d73f0f5d94852da

✔️ pre-commit SUCCESS in 2m 18s
❌ deployment-tests RETRY_LIMIT in 7m 42s

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/99d5e8b0313e49f4be87559073a28c66

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 23s

@mfocko mfocko force-pushed the main branch 2 times, most recently from 6eea607 to 3b29dfa Compare January 21, 2024 19:36
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/26252f142f1944c08c8e41d2c4c6f72a

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 26s

majamassarini and others added 7 commits February 5, 2024 22:14
@majamassarini @mfocko
Do not use anymore the fluentd sidecar pod
a0bc81f
@mfocko
fix: use image stream for nginx, pushgateway
b88571b 
When pulling the images from ghcr.io it is easy to reach the quotas /
rate limiting, therefore create an image stream so it is possible to
manually push the image to the MP+ registry.

TODO by Jirka: research the possibility of mirroring the images into
separate image registry dedicated to MP+.

Signed-off-by: Matej Focko <[email protected]>
@mfocko
fix: change to MP+ specific values
8ecc1e8 
* Sandcastle uses different storage class on MP+
* We also use newer postgres on mp+

Signed-off-by: Matej Focko <[email protected]>
@mfocko
fix(mpp): correct Sandcastle variables
36eaec5 
* namespace
* storage type

Signed-off-by: Matej Focko <[email protected]>
@mfocko @jpopelka
fix: remove the notes about OpenShift Online
65adce4 
Co-authored-by: Jiří Popelka <[email protected]>
Signed-off-by: Matej Focko <[email protected]>
@mfocko
feat(mpp): auto-import the images
7ef37d2 
tl;dr Production gets redeployed on each image build from `stable`

After migration to MP+, we're switching to automatic pulling of the
images by the OpenShift. This process allows us to monitor the
production deployment during the working hours as opposed to the
previous deployment at 2am and verification tests being run at the 4am.

Signed-off-by: Matej Focko <[email protected]>
@mfocko
fix: remove the Automotive production URL
4110dcc 
Signed-off-by: Matej Focko <[email protected]>
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/81b3f39ac3de452c8a39c3b49c7da484

✔️ pre-commit SUCCESS in 1m 42s
❌ deployment-tests RETRY_LIMIT in 7m 44s

@mfocko mfocko self-assigned this Feb 5, 2024
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/72637aa627fa4992be47488b79b127ed

✔️ pre-commit SUCCESS in 1m 41s
❌ deployment-tests RETRY_LIMIT in 7m 35s

majamassarini
majamassarini approved these changes Feb 6, 2024
View reviewed changes
Copy link
Member

@majamassarini majamassarini left a comment

Choose a reason for hiding this comment

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

For what I can grasp this looks good to me.
Thanks a lot!

@mfocko mfocko merged commit 4f8102c into packit:main Feb 6, 2024
2 of 3 checks passed
@mfocko mfocko deleted the mpp-prod-move branch February 6, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nforro nforro nforro left review comments

@jpopelka jpopelka jpopelka left review comments

@majamassarini majamassarini majamassarini approved these changes

Assignees

@mfocko mfocko

Labels
None yet
Projects
Packit Kanban Board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mfocko @jpopelka @majamassarini @nforro