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

Correct traffic_class_name population from tclass_names #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davegreen
Copy link

Fixes error raised in #10

Copy link
Collaborator

@adamwilkins-hpe adamwilkins-hpe left a comment

Choose a reason for hiding this comment

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

I don't have a current environment to test the telemetry demo to test the changes you suggest. Please update your local copy repo for now until someone can test this change. Thanks for contributing!

@alexamayahpe
Copy link
Collaborator

@adamwilkins-hpe, are you asking if someone who has pyedgeconnect running can validate these changes before merging? Or it has been tested and we need to approve to merge into the development branch.

@adamwilkins-hpe
Copy link
Collaborator

@alexamayahpe No, this is about the telemetry demo environment. I have pyedgeconnect, but there is quite a bit of logical infrastructure required for the telemetry demo e.g. Docker, Grafana, InfluxDB, and Redis. As a result, I cannot test anything to do with that at this time.

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.

None yet

3 participants