Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Extend the UC testing workflow to add config provider snap #178

Merged
merged 6 commits into from
Apr 4, 2023
Merged

Extend the UC testing workflow to add config provider snap #178

merged 6 commits into from
Apr 4, 2023

Conversation

MonicaisHer
Copy link
Contributor

Extend the UC testing workflow to add config provider snap

@MonicaisHer
Extend the UC testing workflow to add config provider snap
76f066d 
Extend the UC testing workflow to add config provider snap
@MonicaisHer
Update the format
de6959f 
- disable the security feature of edgexfoundry in V2 format
- directly modify the 'mmodel.yaml' file
- update the format in the 'test.sh' script
@MonicaisHer
Copy link
Contributor Author

MonicaisHer commented Apr 3, 2023 •
edited
Loading

When I tried to run the image, I encountered errors. To troubleshoot the issue, I enabled debugging and found the following errors in the log:

snapd[785]: helpers.go:146: error trying to compare the snap system key: system-key missing on disk

And I find this following error strange because security-secret-store is set to off in build.sh, but the log shows it is set to false which is incorrect:

edgexfoundry.configure[2682]: security service config error: invalid setting for security-secret-store: false

Full debugging logs: https://pastebin.canonical.com/p/GR7y2cg9Kw/

@MonicaisHer MonicaisHer requested a review from farshidtz April 4, 2023 12:01
farshidtz
farshidtz approved these changes Apr 4, 2023
View reviewed changes
Copy link
Member

@farshidtz farshidtz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Tested and it works when excluding the config provider which triggers a known bug.

@farshidtz farshidtz merged commit f4ac775 into canonical:main Apr 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@farshidtz farshidtz farshidtz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MonicaisHer @farshidtz