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

Disable review environment #465

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Disable review environment #465

merged 1 commit into from
Jan 17, 2024

Conversation

voneiden
Copy link
Contributor

The review environment has been broken for quite a while.

The review environment has been broken for quite a while.
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d75eb2f) 91.25% compared to head (9850f81) 91.25%.
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #465   +/-   ##
========================================
  Coverage    91.25%   91.25%           
========================================
  Files           55       55           
  Lines         2800     2800           
  Branches       329      329           
========================================
  Hits          2555     2555           
  Misses         189      189           
  Partials        56       56           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@voneiden voneiden marked this pull request as ready for review December 27, 2023 10:58
@voneiden voneiden requested a review from a team December 27, 2023 10:58
Copy link
Contributor

@akikoskinen akikoskinen left a comment

Choose a reason for hiding this comment

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

How about just removing the whole workflow?

@voneiden
Copy link
Contributor Author

Mm not yet. I don't want to hide any evidence 2 days before the full maintenance responsibility shifts to ratkis. There's also the staging environment and gitlab references that need to be checked by Joku ™️ . If they're on longer in use and have been replaced by platta equivalents, then maybe there's further cleaning up to do in the Kubernetes.

I suppose you've not used these review and stage environments that kolga spins up at any point, or?

@akikoskinen
Copy link
Contributor

Mm not yet. I don't want to hide any evidence 2 days before the full maintenance responsibility shifts to ratkis. There's also the staging environment and gitlab references that need to be checked by Joku ™️ . If they're on longer in use and have been replaced by platta equivalents, then maybe there's further cleaning up to do in the Kubernetes.

I suppose you've not used these review and stage environments that kolga spins up at any point, or?

These haven't been used for years, AFAIK. These (try to) deploy to some KuVa infrastructure somewhere. I'm surprised that any of them (the staging workflow at least) have succeeded during the last couple of years.

If this is the GitLab pipeline, it seems that it's not used either anymore.

There are no direct replacements. There hasn't been a need for review environments. develop branch does get automatically deployed to Platta DEV environment, but that's taken care of in Azure, not in this repository.

Copy link
Contributor

@charn charn left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 :shipit:

@voneiden voneiden merged commit adbbcf8 into develop Jan 17, 2024
6 checks passed
@voneiden voneiden deleted the disable-review branch January 17, 2024 13:11
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