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

Upgrade snap base to core22 #15

Merged
merged 1 commit into from
May 15, 2023
Merged

Upgrade snap base to core22 #15

merged 1 commit into from
May 15, 2023

Conversation

farshidtz
Copy link
Member

@farshidtz farshidtz commented May 11, 2023

Upgrade the snap base to core22 and replace the deprecated environment variables.

@MonicaisHer
Copy link
Contributor

The snap testing doesn't catch the max retries error from device-virtual:

    config.go:237: Time out: reached max 10 retries.
    exec.go:19: [exec] (sudo journalctl --since "2023-05-11 16:06:18" --no-pager | grep "edgex-device-virtual"|| true) > edgex-device-virtual.log
Wrote snap logs to /home/runner/work/_actions/canonical/edgex-snap-testing/v3/test/suites/edgex-config-provider/edgex-device-virtual.log
    exec.go:19: [exec] sudo snap remove --purge edgex-device-virtual
    exec.go:101: [stdout] edgex-device-virtual removed
--- PASS: TestConfigProvider (28.63s)
    --- PASS: TestConfigProvider/device-virtual (28.63s)
May 11 16:06:32 fv-az268-845 edgex-device-virtual.device-virtual[5247]: level=ERROR ts=2023-05-11T16:06:32.569694095Z app=device-virtual source=init.go:95 msg="Failed to load devices: request failed, status code: 400, err: {\"apiVersion\":\"v3\",\"message\":\"AddDeviceRequest.Device.AutoEvents[0].Interval field is required; AddDeviceRequest.Device.AutoEvents[0].SourceName field is required\",\"statusCode\":400}\n"

An issue has been opened: canonical/edgex-snap-testing#206

@farshidtz
Copy link
Member Author

Rebased to include config file changes.

Copy link
Contributor

@MonicaisHer MonicaisHer left a comment

Choose a reason for hiding this comment

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

Thank you, the test has now passed.

@farshidtz farshidtz merged commit 1778bd4 into main May 15, 2023
2 checks passed
@farshidtz farshidtz deleted the core22 branch May 15, 2023 14:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants