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

Remove references to 'iotedged' from iotedge help text #4740

Merged
merged 2 commits into from
Apr 1, 2021

Conversation

daprilik
Copy link
Contributor

@arsing grepping for iotedged in iotedge includes references to "iotedged workload ca my-device". Should this be renamed to something else as well?

@arsing
Copy link
Member

arsing commented Mar 31, 2021

It's the auto-generated Edge CA's CN that @massand reused from the old workload CA. We can change it to aziot-edge CA ...

@kodiakhq kodiakhq bot merged commit 0f82c62 into Azure:master Apr 1, 2021
@daprilik daprilik deleted the bug/iotedge-check-wording branch April 1, 2021 17:04
kodiakhq bot pushed a commit that referenced this pull request Jan 14, 2022
This affects the certd config generated by running `iotedge config apply`,
as well as a new Edge CA cert issued when the current one expires *and*
the common name has not been set explicitly in certd config already.

Fixes #5937
Ref #4740

Also fix iotedge-check tests to not require sudo access.

Ref #5927 (comment)
lt72 pushed a commit to lt72/iotedge that referenced this pull request Jan 31, 2022
…5998)

This affects the certd config generated by running `iotedge config apply`,
as well as a new Edge CA cert issued when the current one expires *and*
the common name has not been set explicitly in certd config already.

Fixes Azure#5937
Ref Azure#4740

Also fix iotedge-check tests to not require sudo access.

Ref Azure#5927 (comment)
damonbarry pushed a commit to damonbarry/iotedge that referenced this pull request Apr 15, 2022
@arsing grepping for `iotedged` in `iotedge` includes references to `"iotedged workload ca my-device"`. Should this be renamed to something else as well?
damonbarry added a commit to damonbarry/iotedge that referenced this pull request Apr 15, 2022
…5998)

This affects the certd config generated by running `iotedge config apply`,
as well as a new Edge CA cert issued when the current one expires *and*
the common name has not been set explicitly in certd config already.

Fixes Azure#5937
Ref Azure#4740

Also fix iotedge-check tests to not require sudo access.

Ref Azure#5927 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants