Skip to content

Helm deployment#9

Draft
kohpai wants to merge 12 commits into
mainfrom
feature/helm
Draft

Helm deployment#9
kohpai wants to merge 12 commits into
mainfrom
feature/helm

Conversation

@kohpai
Copy link
Copy Markdown
Contributor

@kohpai kohpai commented Feb 22, 2025

  • Upgrade express to 5.0.1
  • Fix SQL syntax errors
  • Change default port from 5000 to 8080 to avoid collision with Mac's internal services
  • Add Helm chart for deployment in Kubernetes

@kohpai kohpai requested a review from oat9002 February 22, 2025 16:16
@oat9002
Copy link
Copy Markdown
Contributor

oat9002 commented Feb 22, 2025

Where is helm chart?

@kohpai
Copy link
Copy Markdown
Contributor Author

kohpai commented Feb 23, 2025

Where is helm chart?

It's still a draft MR, lol. Not yet there.

@kohpai
Copy link
Copy Markdown
Contributor Author

kohpai commented Feb 23, 2025

@oat9002 Do you know this error? It occurs after I run all the SQL in sql/ directory, then I run yarn run dev. Before I executed the SQL scripts, the application had run fine.

Error: A naming conflict has occurred - two entities have tried to define the same key 'fromAccount'.

  The first entity was:

    Forward relation for constraint "transaction_from_account_id_fkey" on table "public"."transaction". To rename this relation with a 'Smart Comment':
    
      COMMENT ON CONSTRAINT "transaction_from_account_id_fkey" ON "public"."transaction" IS E'@fieldName newNameHere';

  The second entity was:

    Adding computed column for function "public"."transaction_fromAccount"(...args...). You can rename this field with a 'Smart Comment':
    
      COMMENT ON FUNCTION "public"."transaction_fromAccount"(...arg types go here...) IS E'@fieldName newNameHere';
    at Object.extend (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphile-build/src/extend.js:33:13)
    at hook (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphile-build-pg/src/plugins/PgComputedColumnsPlugin.js:81:14)
    at SchemaBuilder.applyHooks (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphile-build/src/SchemaBuilder.js:411:20)
    at fields (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphile-build/src/makeNewBuild.js:665:40)
    at resolveThunk (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/definition.js:480:40)
    at defineFieldMap (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/definition.js:692:18)
    at GraphQLObjectType.getFields (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/definition.js:633:27)
    at collectReferencedTypes (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/schema.js:366:81)
    at collectReferencedTypes (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/schema.js:368:9)
    at collectReferencedTypes (/Users/kohpai/Projects/ExpenseLedger/api-v2/node_modules/graphql/type/schema.js:368:9)

error Command failed with exit code 34.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

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.

2 participants