Skip to content

Conversation

@samuelallan72
Copy link

@samuelallan72 samuelallan72 commented Sep 12, 2024

Note: we cannot add functional tests for noble, because this is a subordinate charm and we don't have any other principle charms that support noble yet.

I did add functional tests for focal; they were missing from CI.

@samuelallan72
Copy link
Author

I think this is failing because the workflow is using charmcraft 3, and charmcraft 3 maybe only supports noble using the new syntax? I've opened canonical/solutions-engineering-automation#79 to update this repo with our latest check template which pins charmcraft to v2.

Pjack
Pjack previously approved these changes Sep 13, 2024
aieri
aieri previously approved these changes Sep 13, 2024
@samuelallan72
Copy link
Author

Hmm actually there must be something wrong with the check.yaml workflow, because it's not running... I wish github gave an error rather than silently ignored broken workflow files 😕

@samuelallan72
Copy link
Author

Reverted the changes to check.yaml.

@samuelallan72
Copy link
Author

Hmm maybe the promtool tests workflow is overriding check.yaml? I checked the PR where I introduced that, and the check.yaml workflow wasn't running there either: #310 :/ We really need to add the list of workflows required to the protection rules.

@samuelallan72
Copy link
Author

They're all running in #311 , so maybe we can merge that first. :)

@samuelallan72 samuelallan72 merged commit e7448ae into main Sep 16, 2024
7 checks passed
@samuelallan72 samuelallan72 deleted the noble branch September 16, 2024 09:44
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.

5 participants