You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We've encountered a lot of instability in the kfp CI of late, including oddly flaky charmcraft builds for some of the charms. For some unknown reasons, standalone charmcraft seems to build things more reliably than building them in the pytest file directly using the pytest-operator.
Proposed here is that we move the building for these charms out of the pytest files and into their own jobs, then pass the .charm files as artifacts to the tests/publish jobs. This would hopefully add stability just from using charmcraft, but also we can have retry logic around failed build jobs (build jobs could auto-retry X times, so flakes could be mitigated)
What needs to get done
refactor the pytest tests to optionally build the charms or accept their artifacts as inputs. We might be able to use some data platform CI as inspiration here
refactor the CI workflows to build the charms separately and pass them to integration tests
Definition of Done
kfp integration test CI does not build its own charms, but instead uses those built in separate jobs
The text was updated successfully, but these errors were encountered:
Context
We've encountered a lot of instability in the kfp CI of late, including oddly flaky charmcraft builds for some of the charms. For some unknown reasons, standalone charmcraft seems to build things more reliably than building them in the pytest file directly using the pytest-operator.
Proposed here is that we move the building for these charms out of the pytest files and into their own jobs, then pass the .charm files as artifacts to the tests/publish jobs. This would hopefully add stability just from using charmcraft, but also we can have retry logic around failed build jobs (build jobs could auto-retry X times, so flakes could be mitigated)
What needs to get done
Definition of Done
The text was updated successfully, but these errors were encountered: