Skip to content

Conversation

saeedfazal
Copy link
Contributor

What changed?

  • Added Preview verification for applications and capi cluster PRs
  • Speed up tests by force reconciling flux to install applications and clusters

Why was this change made?
To increase test coverage

How was this change implemented?

  • Updating existing tests to add preview verification

How did you validate the change?
The tests are passing locally

@saeedfazal saeedfazal self-assigned this Nov 7, 2022
@saeedfazal saeedfazal force-pushed the add-preview-acceptance-test branch 2 times, most recently from 3fc0bfb to ebb1cbe Compare November 8, 2022 16:55
}
}
return fileList, nil
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice :)

Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

💯

@saeedfazal saeedfazal force-pushed the add-preview-acceptance-test branch from 604cf7f to 1956234 Compare November 10, 2022 10:06
@saeedfazal saeedfazal force-pushed the add-preview-acceptance-test branch from 1956234 to 47062a5 Compare November 10, 2022 13:21
@saeedfazal saeedfazal merged commit 6b0c1e2 into main Nov 10, 2022
@saeedfazal saeedfazal deleted the add-preview-acceptance-test branch November 10, 2022 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants