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

remove "SharedVolumeService" design and "Uploading" status #5

Merged
merged 18 commits into from
May 21, 2024
Merged

Conversation

jim60105
Copy link
Member

@jim60105 jim60105 commented May 21, 2024

SharedVolumeService design always causes problems in K8s, and increasing unnecessary expenses on Azure.

jim60105 and others added 7 commits May 21, 2024 03:23
Signed-off-by: 陳鈞 <jim60105@gmail.com>
Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Added a new GitHub Action for publishing Docker images on dev branch pushes
- This action, named "docker_publish_dev", triggers on 'push' event specifically for 'dev' branch only
- Also included provision for manual workflow triggering with the use of 'workflow_dispatch'
- The job sequence includes checking out the code, setting Docker meta for Apache CouchDB and Azure CosmosDB, logging in to DockerHub and GitHub Container Registry, and building and pushing respective Docker images
- Moved the Docker images to ${{ secrets.DOCKERHUB_ORGANIZATION_NAME }}/${{ github.event.repository.name }} and ghcr.io/${{ github.repository }}
- Utilized Docker actions for login and metadata generation, and for building and pushing images.

Signed-off-by: 陳鈞 <jim60105@gmail.com>
… LivestreamRecorderService

- Change the `sync_ref` to target the `dev` branch instead of `master` in the `update-submodule` github action.

Signed-off-by: 陳鈞 <jim60105@gmail.com>
k8s.Autorest.HttpOperationException: Operation returned an invalid status code 'UnprocessableEntity', response body {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Job.batch \"ytdlpyvfcalhoalx0\" is invalid: spec.template.spec.containers[0].name: Invalid value: \"ytdlpyvfcalhoalx0_uploader\": a lowercase RFC 1123 label must consist of lower case alphanumeric characters or '-', and must start and end with an alphanumeric character (e.g. 'my-name',  or '123-abc', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?')","reason":"Invalid","details":{"name":"ytdlpyvfcalhoalx0","group":"batch","kind":"Job","causes":[{"reason":"FieldValueInvalid","message":"Invalid value: \"ytdlpyvfcalhoalx0_uploader\": a lowercase RFC 1123 label must consist of lower case alphanumeric characters or '-', and must start and end with an alphanumeric character (e.g. 'my-name',  or '123-abc', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?')","field":"spec.template.spec.containers[0].name"}]},"code":422}

Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Changed the argument path in `KubernetesServiceBase` to be filename only instead of a combination of mount path and filename
- Fixed hardcoded 'MountPath' to be "/sharedvolume" in `KubernetesServiceBase` instead of dynamic mount path

Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Added a new `cookies` volume to the Kubernetes service base configuration, sourced from a named secret.
- Made the `cookies` volume optional with a default mode set to 660.
- Created a mounting path for the `cookies` volume in the init containers configuration, keeping the property as not read-only.

Signed-off-by: 陳鈞 <jim60105@gmail.com>
Copy link
Contributor

deepsource-io bot commented May 21, 2024

Here's the code health analysis summary for commits 762f0ca..0aaab8a. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource C# LogoC#❌ Failure
❗ 2 occurences introduced
🎯 2 occurences resolved
View Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

jim60105 and others added 8 commits May 21, 2024 16:42
Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Modify the handling of channel and video Ids for YouTube, Twitch, Twitcasting, and FC2 to better handle prefix changes for these platforms in `NameHelper.cs`
- Correct the URL formation process in `YtdlpService.cs` to remove the unnecessary Id handling, thus directly appending `url` to the YouTube base URL

Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Mark `UpdateVideosDataAsync` function in `Fc2Service` for future refactoring due to complexity
- Mark `UpdateVideoDataAsync` function in `YoutubeService` for future refactoring due to complexity

Signed-off-by: 陳鈞 <jim60105@gmail.com>
Signed-off-by: 陳鈞 <jim60105@gmail.com>
Signed-off-by: 陳鈞 <jim60105@gmail.com>
Properties should not return arrays as they are not write protected CS-W1096

Signed-off-by: 陳鈞 <jim60105@gmail.com>
…hDB.NET and EF Core Cosmos

- Update `CouchDB.NET` package from version `3.6.0` to `3.6.1`
- Update `Microsoft.EntityFrameworkCore.Cosmos` package from version `6.0.29` to `6.0.30`

Signed-off-by: 陳鈞 <jim60105@gmail.com>
- Bumped up the versions of several dependencies, such as `Azure.Identity`, `Azure.ResourceManager`, `Azure.ResourceManager.ContainerInstance`, `Azure.ResourceManager.Resources`, `Azure.Storage.Blobs`, `CouchDB.NET.DependencyInjection`, `Discord.Net.Webhook`, `KubernetesClient`, `Serilog.Sinks.Seq`, `Microsoft.EntityFrameworkCore.Cosmos`, and `Microsoft.EntityFrameworkCore.Design`.

Signed-off-by: 陳鈞 <jim60105@gmail.com>
Signed-off-by: 陳鈞 <jim60105@gmail.com>
jim60105 added 2 commits May 22, 2024 02:35
… must be lowercase

Error: buildx failed with: ERROR: failed to solve: failed to configure registry cache exporter: invalid reference format: repository name (Recorder-moe/LivestreamRecorderService) must be lowercase
Signed-off-by: 陳鈞 <jim60105@gmail.com>
Signed-off-by: 陳鈞 <jim60105@gmail.com>
@jim60105 jim60105 marked this pull request as ready for review May 21, 2024 18:51
@jim60105 jim60105 merged commit c491158 into master May 21, 2024
4 of 5 checks passed
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.

1 participant