-
Notifications
You must be signed in to change notification settings - Fork 1
[elixir] feat: add admin app #116
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,211 @@ | ||
| # This file contains the configuration for Credo and you are probably reading | ||
| # this after creating it with `mix credo.gen.config`. | ||
| # | ||
| # If you find anything wrong or unclear in this file, please report an | ||
| # issue on GitHub: https://github.com/rrrene/credo/issues | ||
| # | ||
| %{ | ||
| # | ||
| # You can have as many configs as you like in the `configs:` field. | ||
| configs: [ | ||
| %{ | ||
| # | ||
| # Run any config using `mix credo -C <name>`. If no config name is given | ||
| # "default" is used. | ||
| # | ||
| name: "default", | ||
| # | ||
| # These are the files included in the analysis: | ||
| files: %{ | ||
| # | ||
| # You can give explicit globs or simply directories. | ||
| # In the latter case `**/*.{ex,exs}` will be used. | ||
| # | ||
| included: [ | ||
| "config/", | ||
| "lib/", | ||
| "src/", | ||
| "test/", | ||
| "web/", | ||
| "apps/*/lib/", | ||
| "apps/*/src/", | ||
| "apps/*/test/", | ||
| "apps/*/web/" | ||
| ], | ||
| excluded: [~r"/_build/", ~r"/deps/", ~r"/node_modules/"] | ||
| }, | ||
| # | ||
| # Load and configure plugins here: | ||
| # | ||
| plugins: [], | ||
| # | ||
| # If you create your own checks, you must specify the source files for | ||
| # them here, so they can be loaded by Credo before running the analysis. | ||
| # | ||
| requires: [], | ||
| # | ||
| # If you want to enforce a style guide and need a more traditional linting | ||
| # experience, you can change `strict` to `true` below: | ||
| # | ||
| strict: true, | ||
| # | ||
| # To modify the timeout for parsing files, change this value: | ||
| # | ||
| parse_timeout: 5000, | ||
| # | ||
| # If you want to use uncolored output by default, you can change `color` | ||
| # to `false` below: | ||
| # | ||
| color: true, | ||
| # | ||
| # You can customize the parameters of any check by adding a second element | ||
| # to the tuple. | ||
| # | ||
| # To disable a check put `false` as second element: | ||
| # | ||
| # {Credo.Check.Design.DuplicatedCode, false} | ||
| # | ||
| checks: %{ | ||
| enabled: [ | ||
| # | ||
| ## Consistency Checks | ||
| # | ||
| {Credo.Check.Consistency.ExceptionNames, []}, | ||
| {Credo.Check.Consistency.LineEndings, []}, | ||
| {Credo.Check.Consistency.ParameterPatternMatching, []}, | ||
| {Credo.Check.Consistency.SpaceAroundOperators, []}, | ||
| {Credo.Check.Consistency.SpaceInParentheses, []}, | ||
| {Credo.Check.Consistency.TabsOrSpaces, []}, | ||
| {Credo.Check.Consistency.MultiAliasImportRequireUse, []}, | ||
| {Credo.Check.Consistency.UnusedVariableNames, []}, | ||
|
|
||
| # | ||
| ## Design Checks | ||
| # | ||
| # You can customize the priority of any check | ||
| # Priority values are: `low, normal, high, higher` | ||
| # | ||
| {Credo.Check.Design.AliasUsage, | ||
| [priority: :low, if_nested_deeper_than: 2, if_called_more_often_than: 0]}, | ||
| # You can also customize the exit_status of each check. | ||
| # If you don't want TODO comments to cause `mix credo` to fail, just | ||
| # set this value to 0 (zero). | ||
| # | ||
| {Credo.Check.Design.TagTODO, [exit_status: 2]}, | ||
| {Credo.Check.Design.TagFIXME, []}, | ||
| {Credo.Check.Design.DuplicatedCode, []}, | ||
| {Credo.Check.Design.SkipTestWithoutComment, []}, | ||
|
|
||
| # | ||
| ## Readability Checks | ||
| # | ||
| {Credo.Check.Readability.AliasOrder, []}, | ||
| {Credo.Check.Readability.FunctionNames, []}, | ||
| {Credo.Check.Readability.LargeNumbers, []}, | ||
| # matches 'py_cubic_ingestion/pyprojest.toml' | ||
| {Credo.Check.Readability.MaxLineLength, [priority: :low, max_length: 120]}, | ||
| {Credo.Check.Readability.ModuleAttributeNames, []}, | ||
| {Credo.Check.Readability.ModuleDoc, []}, | ||
| {Credo.Check.Readability.ModuleNames, []}, | ||
| {Credo.Check.Readability.ParenthesesInCondition, []}, | ||
| {Credo.Check.Readability.ParenthesesOnZeroArityDefs, []}, | ||
| {Credo.Check.Readability.PipeIntoAnonymousFunctions, []}, | ||
| {Credo.Check.Readability.PredicateFunctionNames, []}, | ||
| {Credo.Check.Readability.PreferImplicitTry, []}, | ||
| {Credo.Check.Readability.RedundantBlankLines, []}, | ||
| {Credo.Check.Readability.Semicolons, []}, | ||
| {Credo.Check.Readability.SpaceAfterCommas, []}, | ||
| {Credo.Check.Readability.StringSigils, []}, | ||
| {Credo.Check.Readability.TrailingBlankLine, []}, | ||
| {Credo.Check.Readability.TrailingWhiteSpace, []}, | ||
| {Credo.Check.Readability.UnnecessaryAliasExpansion, []}, | ||
| {Credo.Check.Readability.VariableNames, []}, | ||
| {Credo.Check.Readability.WithSingleClause, []}, | ||
| {Credo.Check.Readability.AliasAs, []}, | ||
| {Credo.Check.Readability.BlockPipe, []}, | ||
| {Credo.Check.Readability.ImplTrue, []}, | ||
| {Credo.Check.Readability.MultiAlias, []}, | ||
| {Credo.Check.Readability.NestedFunctionCalls, false}, | ||
| {Credo.Check.Readability.SeparateAliasRequire, []}, | ||
| {Credo.Check.Readability.SingleFunctionToBlockPipe, []}, | ||
| {Credo.Check.Readability.SinglePipe, []}, | ||
| {Credo.Check.Readability.Specs, []}, | ||
| {Credo.Check.Readability.StrictModuleLayout, []}, | ||
| {Credo.Check.Readability.WithCustomTaggedTuple, []}, | ||
|
|
||
| # | ||
| ## Refactoring Opportunities | ||
| # | ||
| {Credo.Check.Refactor.Apply, []}, | ||
| {Credo.Check.Refactor.CondStatements, []}, | ||
| {Credo.Check.Refactor.CyclomaticComplexity, []}, | ||
| {Credo.Check.Refactor.FunctionArity, []}, | ||
| {Credo.Check.Refactor.LongQuoteBlocks, []}, | ||
| {Credo.Check.Refactor.MatchInCondition, []}, | ||
| {Credo.Check.Refactor.MapJoin, []}, | ||
| {Credo.Check.Refactor.NegatedConditionsInUnless, []}, | ||
| {Credo.Check.Refactor.NegatedConditionsWithElse, []}, | ||
| {Credo.Check.Refactor.Nesting, []}, | ||
| {Credo.Check.Refactor.UnlessWithElse, []}, | ||
| {Credo.Check.Refactor.WithClauses, []}, | ||
| {Credo.Check.Refactor.FilterFilter, []}, | ||
| {Credo.Check.Refactor.RejectReject, []}, | ||
| {Credo.Check.Refactor.RedundantWithClauseResult, []}, | ||
| {Credo.Check.Refactor.ABCSize, []}, | ||
| {Credo.Check.Refactor.AppendSingleItem, []}, | ||
| {Credo.Check.Refactor.DoubleBooleanNegation, []}, | ||
| {Credo.Check.Refactor.FilterReject, []}, | ||
| {Credo.Check.Refactor.IoPuts, []}, | ||
| {Credo.Check.Refactor.MapMap, []}, | ||
| {Credo.Check.Refactor.ModuleDependencies, [max_deps: 15]}, | ||
| {Credo.Check.Refactor.NegatedIsNil, []}, | ||
| {Credo.Check.Refactor.PipeChainStart, []}, | ||
| {Credo.Check.Refactor.RejectFilter, []}, | ||
| {Credo.Check.Refactor.VariableRebinding, []}, | ||
|
|
||
| # | ||
| ## Warnings | ||
| # | ||
| {Credo.Check.Warning.ApplicationConfigInModuleAttribute, []}, | ||
| {Credo.Check.Warning.BoolOperationOnSameValues, []}, | ||
| {Credo.Check.Warning.ExpensiveEmptyEnumCheck, []}, | ||
| {Credo.Check.Warning.IExPry, []}, | ||
| {Credo.Check.Warning.IoInspect, []}, | ||
| {Credo.Check.Warning.OperationOnSameValues, []}, | ||
| {Credo.Check.Warning.OperationWithConstantResult, []}, | ||
| {Credo.Check.Warning.RaiseInsideRescue, []}, | ||
| {Credo.Check.Warning.SpecWithStruct, []}, | ||
| {Credo.Check.Warning.WrongTestFileExtension, []}, | ||
| {Credo.Check.Warning.UnusedEnumOperation, []}, | ||
| {Credo.Check.Warning.UnusedFileOperation, []}, | ||
| {Credo.Check.Warning.UnusedKeywordOperation, []}, | ||
| {Credo.Check.Warning.UnusedListOperation, []}, | ||
| {Credo.Check.Warning.UnusedPathOperation, []}, | ||
| {Credo.Check.Warning.UnusedRegexOperation, []}, | ||
| {Credo.Check.Warning.UnusedStringOperation, []}, | ||
| {Credo.Check.Warning.UnusedTupleOperation, []}, | ||
| {Credo.Check.Warning.UnsafeExec, []}, | ||
| {Credo.Check.Warning.LeakyEnvironment, []}, | ||
| {Credo.Check.Warning.MapGetUnsafePass, []}, | ||
| {Credo.Check.Warning.MixEnv, []}, | ||
| {Credo.Check.Warning.UnsafeToAtom, []} | ||
| ], | ||
| disabled: [ | ||
| # | ||
| # Checks scheduled for next check update (opt-in for now, just replace `false` with `[]`) | ||
|
|
||
| # | ||
| # Controversial and experimental checks (opt-in, just move the check to `:enabled` | ||
| # and be sure to use `mix credo --strict` to see low priority checks) | ||
| # | ||
| {Credo.Check.Warning.LazyLogging, []} | ||
| # {Credo.Check.Refactor.MapInto, []}, | ||
|
|
||
| # | ||
| # Custom checks can be created using `mix credo.gen.check`. | ||
| # | ||
| ] | ||
| } | ||
| } | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| # Used by "mix format" | ||
| [ | ||
| inputs: ["{mix,.formatter}.exs", "{config,lib,test}/**/*.{ex,exs}"] | ||
| ] |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| # The directory Mix will write compiled artifacts to. | ||
| /_build/ | ||
|
|
||
| # If you run "mix test --cover", coverage assets end up here. | ||
| /cover/ | ||
|
|
||
| # The directory Mix downloads your dependencies sources to. | ||
| /deps/ | ||
|
|
||
| # Where third-party dependencies like ExDoc output generated docs. | ||
| /doc/ | ||
|
|
||
| # Ignore .fetch files in case you like to edit your project deps locally. | ||
| /.fetch | ||
|
|
||
| # If the VM crashes, it generates a dump, let's ignore it too. | ||
| erl_crash.dump | ||
|
|
||
| # Also ignore archive artifacts (built via "mix archive.build"). | ||
| *.ez | ||
|
|
||
| # Ignore package tarball (built via "mix hex.build"). | ||
| admin-*.tar | ||
|
|
||
| # Temporary files, for example, from tests. | ||
| /tmp/ | ||
|
|
||
| priv/cert/* |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # Admin | ||
|
|
||
| An Elixir app to get visibility into the Data Platform pipelines and take action on them. | ||
|
|
||
| ## Installation | ||
|
|
||
| 1. Initialize app. | ||
| ```sh | ||
| mix deps.get | ||
| ``` | ||
|
|
||
| 2. Generate SSL certificates. | ||
|
|
||
| ```sh | ||
| mix x509.gen.selfsigned | ||
| ``` | ||
|
|
||
| 3. Run the app. | ||
| ```sh | ||
| iex -S mix | ||
| ``` | ||
|
|
||
| 4. Go to [https://localhost:4000/](https://localhost:4000/) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| import Config | ||
|
|
||
| # Configures Elixir's Logger | ||
| config :logger, :console, | ||
| format: "$time $metadata[$level] $message\n", | ||
| metadata: [:request_id] | ||
|
|
||
| config :admin, | ||
| ecto_repos: [Admin.Repo] | ||
|
|
||
| # Import environment specific config. This must remain at the bottom | ||
| # of this file so it overrides the configuration defined above. | ||
| import_config "#{Mix.env()}.exs" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| import Config | ||
|
|
||
| config :ex_aws, | ||
| # note: 'region' doesn't work. it's inherited from the profile | ||
| # region: [{:system, "AWS_REGION"}, {:awscli, "default", 30}, :instance_role], | ||
| access_key_id: [{:system, "AWS_ACCESS_KEY_ID"}, {:awscli, "default", 30}], | ||
| secret_access_key: [ | ||
| {:system, "AWS_SECRET_ACCESS_KEY"}, | ||
| {:awscli, "default", 30} | ||
| ] | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| import Config | ||
|
|
||
| config :admin, Admin.Repo, | ||
| show_sensitive_data_on_connection_error: false, | ||
| ssl: true, | ||
| use_iam_token: true | ||
|
|
||
| config :ex_aws, | ||
| # overwrite defaults here, so as to only look at instance role | ||
| access_key_id: :instance_role, | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For RDS connection. |
||
| secret_access_key: :instance_role | ||
|
|
||
| config :ehmon, :report_mf, {:ehmon, :info_report} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| import Config | ||
|
|
||
| config :admin, Admin.Repo, | ||
| username: System.get_env("DB_USER"), | ||
| database: System.get_env("DB_NAME"), | ||
| hostname: System.get_env("DB_HOST"), | ||
| password: System.get_env("DB_PASSWORD"), | ||
| port: "DB_PORT" |> System.get_env("5432") |> String.to_integer(), | ||
| configure: {Admin.Repo, :before_connect, []}, | ||
| # default to 10 | ||
| pool_size: "DB_POOL_SIZE" |> System.get_env("10") |> String.to_integer() | ||
|
|
||
| # for testing, we'd like to change the database | ||
| if config_env() == :test do | ||
| config :admin, Admin.Repo, database: "#{System.get_env("DB_NAME")}_test" | ||
| end |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| import Config | ||
|
|
||
| # only log warnings+ in test | ||
| config :logger, level: :info | ||
|
|
||
| config :admin, Admin.Repo, pool: Ecto.Adapters.SQL.Sandbox |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| defmodule Admin.Application do | ||
| # See https://hexdocs.pm/elixir/Application.html | ||
| # for more information on OTP Applications | ||
| @moduledoc false | ||
|
|
||
| use Application | ||
|
|
||
| @impl true | ||
| def start(_type, _args) do | ||
| children = [ | ||
| {Admin.Repo, []}, | ||
| {Plug.Cowboy, | ||
| scheme: :https, | ||
| options: [ | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Question: I need to move this to a config so it can be different on prod, but not sure what the best way to go about it.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. # config/config.exs
config :admin,
cowboy_options: [ scheme: :https, ... ]
# lib/admin/application.ex
{Plug.Cowboy, Application.get_env(:admin, :cowboy_options)} |
||
| certfile: "priv/cert/selfsigned.pem", | ||
| keyfile: "priv/cert/selfsigned_key.pem", | ||
| otp_app: :admin, | ||
| port: 4000 | ||
| ], | ||
| plug: Admin.Router} | ||
| ] | ||
|
|
||
| # See https://hexdocs.pm/elixir/Supervisor.html | ||
| # for other strategies and supported options | ||
| opts = [strategy: :one_for_one, name: Admin.Supervisor] | ||
| Supervisor.start_link(children, opts) | ||
| end | ||
| end | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't need these now, but will eventually.