chore(test): Adds swabbie integration tests for docker builds#669
chore(test): Adds swabbie integration tests for docker builds#669jasonmcintosh wants to merge 9 commits into
Conversation
|
Doesn't this need a chance to some github actions to actually run the tests? |
Yeah :) I'd not gotten all the pieces configured. I should have probably put this as "draft" but hadn't worried too much. I THINK it's actually in a pretty good state now for merge. |
| "${{ env.CONTAINER_REGISTRY }}/${{ steps.build_variables.outputs.REPO }}:${{ github.ref_name }}-latest-java11-unvalidated-ubuntu" | ||
| "${{ env.CONTAINER_REGISTRY }}/${{ steps.build_variables.outputs.REPO }}:${{ steps.build_variables.outputs.VERSION }}-java11-unvalidated-ubuntu" | ||
|
|
||
| - name: Test local slim container image |
There was a problem hiding this comment.
Thanks will add that :)
There was a problem hiding this comment.
And....it fails:
APPLICATION FAILED TO START
***************************
Description:
Config data location '/home/spinnaker/.spinnaker/' does not exist
Action:
Check that the value '/home/spinnaker/.spinnaker/' is correct, or prefix it with 'optional:'
There was a problem hiding this comment.
Yeah :) That's that kork config with the default locations if I recall right. Will dig & fix here later.
There was a problem hiding this comment.
https://github.com/spinnaker/kork/blob/2b857f57c85a8c821e35da68c1207fbf00cb335d/kork-config/src/main/java/com/netflix/spinnaker/kork/boot/DefaultPropertiesBuilder.java#L32C1-L33C1 this was what was thinking. Checking on the config here.
There was a problem hiding this comment.
FYI fixed this with latest PR, but working through all the "clouddriver isn't available" as part of the integration tests. Got some more work to do on the downstream deps.
3 participants
Duplicates most of the other project integration tests with a FEW changes