Bot Sanity Changes #245
Merged
Bot Sanity Changes #245
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is no longer used and can be removed.
Move to .NET Standard 2.0 Only Expose Logger.LogFileBaseDirectory Move the color based logging system to check RuntimeInformation
Remove all Windows build scripts. Move clean.bat to clean.sh Create script to generate an image tag for releases.
This is not needed as the bot is now managed by Nomad.
Removes Arbiter and makes #243 redundant
Add extensions towards CCServiceSoap to BatchJob etc. via Grid.Commands.
ProcessManagement.Core contains shared code across different methods of contacting grid servers. ProcessManagement.Docker.Core contains shared code like a DockerAuthority used to interact with Docker. ProcessManagement.Docker is a PM.Core implementation for Docker.
Update ProcessManagement to use PM.Core
Remove Grid.Shared
Move all classes to class-interface architecture to relate to DI system Remove Exception Based Auto Moderator Move IAttacmentExtenesions to this assembly Move to .NET 6 to accomodate for #150
Move to .NET 6 to accomodate for #150 Move ArbiterSettings to GridSettings and implement docker settings. Rename all "Provider" classes to "Settings" classes. Remove EBAM settings.
Move to .NET 6 to accomodate for #150 Remove AsyncWorkQueues Add slash command executed handlers. Use Native Discord.Net interactions. Update handlers to accommodate for DI.
Removal of custom commands and slash commands in favour of native system. Move to .NET 6 to accomodate for #150 Move to support DI.
Move from services/grid-bot/src to /src Move to .NET 6 to accomodate for #150 Move to native Discord.Net commands. Support dependency injection. Remove run service scripts.
nikita-petko
added
opsec: love-all-platforms
Contributor
Author
|
This will be deferred temporarily until the following are finished:
|
build.yml: - Remove pull_request check. - Make NOMAD_JOB_NAME append -dev or -prod depending on NOMAD_ENVIRONMENT - Make version append -dev if building Debug - Change directory into publish dir. - Remove Docker BuildKit ENVVAR - Remove requirement for NOMAD_TOKEN - Change sed delimiter for IMAGE_NAME so it doesn't conflct codeql.yml: - Fix condition syntax
- Skip if commit message contains #!skip-build-and-deploy!# - Change default registries depending on branch.
Change vault policy. Add 2 container mounts Change default log file directory Signed-off-by: Nikita Petko <petko@vmminfra.net>
Fix default log file directory Signed-off-by: Nikita Petko <petko@vmminfra.net>
Change env var name Signed-off-by: Nikita Petko <petko@vmminfra.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #238
Fixes #237
Fixes #236
Fixes #235
Fixes #234
Fixes #210
Fixes #155
Fixes #150
Fixes #144
Fixes #114
Fixes #111
Fixes #52