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

[BUG] OpenSearch Reporting CLI Lambda not evaluating certain environment variables #38

Closed
aaronbrighton opened this issue Apr 6, 2023 · 2 comments
Assignees
Labels
bug Something isn't working v1.1.0

Comments

@aaronbrighton
Copy link

What is the bug?

When using the OpenSearch Reporting CLI Lambda function, the following environment variables are not evaluated correctly. Instead they are using the default values.

  • OPENSEARCH_EMAIL_SUBJECT
  • OPENSEARCH_EMAIL_NOTE

How can one reproduce the bug?

Follow the instructions here (https://opensearch.org/docs/latest/dashboards/reporting-cli/rep-cli-lambda/) and set any of the above environment variables on the Lambda function instead of passing them as the event object.

What is the expected behavior?

Expectation is that the email sent by the Lambda function would contain a non-default subject and note.

What is your host/environment?

N/A

Do you have any screenshots?

See description for observed behavior.

Do you have any additional context?

It appears the following code function causes bypass of the environment variables:

async function getEventArguments(event) {

@aaronbrighton aaronbrighton added bug Something isn't working untriaged labels Apr 6, 2023
@rupal-bq rupal-bq removed the untriaged label Apr 6, 2023
@rupal-bq
Copy link
Member

rupal-bq commented Apr 6, 2023

@aaronbrighton Thanks for reporting the issue. I will take a look.

@rupal-bq
Copy link
Member

Hi @aaronbrighton, this will be fixed in next release v1.1.0.

If you would like to try/test before the release, you can refer to doc for building docker image from source code https://github.com/opensearch-project/reporting-cli/blob/main/docs/examples/lambda_container_image_from_source.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v1.1.0
Projects
None yet
Development

No branches or pull requests

2 participants