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

Fix/update env variables #242

Merged
merged 3 commits into from
Jul 25, 2024
Merged

Fix/update env variables #242

merged 3 commits into from
Jul 25, 2024

Conversation

adev4a
Copy link
Collaborator

@adev4a adev4a commented Jul 24, 2024

PR Details

Updates env variables to allow loading of env file for telematics units as well as fixes retention policy for local historical data processing bucket.

Description

Related GitHub Issue

Related Jira Key

Motivation and Context

How Has This Been Tested?

Types of changes

  • Defect fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that cause existing functionality to change)

Checklist:

  • I have added any new packages to the sonar-scanner.properties file
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@adev4a adev4a requested a review from dan-du-car July 24, 2024 14:58
@adev4a adev4a changed the base branch from develop to release/neon July 24, 2024 14:58
@adev4a adev4a self-assigned this Jul 24, 2024
MESSAGING_INFLUX_URI= #Required: public IP for machine where influxdb is hosted
MESSAGING_INFLUX_PORT= #Required: Port at which influxdb is available
MESSAGING_INFLUX_USERNAME= #Required: Username for influxdb instance
MESSAGING_INFLUX_PWD= #Required: #Required: Password for influxdb instance
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It has two required

@@ -88,13 +88,13 @@ FILE_EXTENSIONS=.mcap # Only query a list of objects with supported file extensi
# Messaging Server Config
### Log level (INFO, DEBUG, ERROR)
MESSAGING_LOGGING_LEVEL=INFO
MESSAGING_NATS_URI=nats://<NATS_IP>:4222
MESSAGING_NATS_URI= #change to amazon ec2 ipv4 address:nats_port
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe provide an example ?

@dan-du-car dan-du-car merged commit a09ef47 into release/neon Jul 25, 2024
16 checks passed
@dan-du-car dan-du-car deleted the fix/update_env_variables branch July 25, 2024 13:36
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