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

tests: update spread test for unknown plug/slot with snapctl is-connected #9172

Merged
merged 3 commits into from
Aug 21, 2020

Conversation

stolowski
Copy link
Contributor

Update snapctl is-connected spread test to check new error introduced with #9168 (will fail until that PR lands).

@stolowski stolowski added Simple 😃 A small PR which can be reviewed quickly Test Robustness labels Aug 18, 2020
@sergiocazzolato
Copy link
Collaborator

sergiocazzolato commented Aug 19, 2020

@stolowski, thanks for the update

The test is failing on debian with this:

++ test-snap.checkconn home

  • OUT=
  • MATCH 'error running snapctl: snap "test-snap" has no plug or slot named "home"'
  • echo ''
    grep error: pattern not found, got:

@sergiocazzolato
Copy link
Collaborator

Also saw this in fedora:

++ test-snap.checkconn home

  • OUT='WARNING: cgroup v2 is not fully supported yet, proceeding with partial confinement'
  • MATCH 'error running snapctl: snap "test-snap" has no plug or slot named "home"'
  • echo 'WARNING: cgroup v2 is not fully supported yet, proceeding with partial confinement'
    grep error: pattern not found, got:
    WARNING: cgroup v2 is not fully supported yet, proceeding with partial confinement

@stolowski
Copy link
Contributor Author

@stolowski, thanks for the update

The test is failing on debian with this:

As mentioned in the description failures were expected until the prerequisite lands; this just happened and I merged master, let's see how it goes now.

@stolowski stolowski merged commit 080734a into canonical:master Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Simple 😃 A small PR which can be reviewed quickly Test Robustness
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants