-
Notifications
You must be signed in to change notification settings - Fork 468
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
Fixups from #3324 #3419
Fixups from #3324 #3419
Conversation
Fixes some bugs and other issues, including unnecessary metadata options and uneven info in `metadata.yaml` files, introduced in #3324 Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
FYI @sicoyle |
reviewing rn. Thanks for the tag here :) |
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.
few minor things. Thanks for the adjustments and cleanup :)
The connection string for the PostgreSQL database | ||
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | ||
example: | | ||
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" | ||
type: string |
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.
The connection string for the PostgreSQL database | |
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | |
example: | | |
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" | |
type: string | |
The connection string for the PostgreSQL database. | |
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | |
example: | | |
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" | |
type: string |
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.
Thanks. Note that this section was in the metadata.yaml
for 2 of the 4 components already, but was missing on 2
The connection string for the PostgreSQL database | ||
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | ||
example: | | ||
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" |
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.
The connection string for the PostgreSQL database | |
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | |
example: | | |
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" | |
The connection string for the PostgreSQL database. | |
This must contain the user, which corresponds to the name of the user created inside PostgreSQL that maps to the AWS IAM policy. This connection string should not contain any password. Note that the database name field is denoted by dbname with AWS. | |
example: | | |
"host=mydb.postgres.database.aws.com user=myapplication port=5432 dbname=dapr_test sslmode=require" |
@holopin-bot @ItalyPaleAle Enjoy the Dapr components badge! |
Congratulations @ItalyPaleAle, you just earned a badge! Here it is: https://holopin.io/claim/clxj3sppo29590dlgav7fwh6v This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Fixes some bugs and other issues, including unnecessary metadata options and uneven info in
metadata.yaml
files, introduced in #3324