Skip to content

test: add backwards compatibility test setup [WIP]#1065

Merged
ryanzhang-oss merged 10 commits intoAzure:mainfrom
michaelawyu:test/backwards-compatibility
Mar 4, 2025
Merged

test: add backwards compatibility test setup [WIP]#1065
ryanzhang-oss merged 10 commits intoAzure:mainfrom
michaelawyu:test/backwards-compatibility

Conversation

@michaelawyu
Copy link
Contributor

Description of your changes

This PR adds backwards compatibility tests to the Fleet repo to verify if the system would perform as expected when:

a) a new hub agent working with an old member agent;
b) an old hub agent working with a new member agent;
c) new Fleet agents (hub + member) processing objects created by old Fleet agents

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

N/A

Special notes for your reviewer

WIP.


// checkIfPlacedWorkResourcesOnMemberClustersConsistently verifies if the work resources have been placed on
// the specified set of member clusters.
func checkIfPlacedWorkResourcesOnMemberClustersConsistently(clusters []*framework.Cluster, workNamespaceName, appConfigMapName string) {
Copy link
Contributor

Choose a reason for hiding this comment

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

are those copy from the e2e lib?

Copy link
Contributor

Choose a reason for hiding this comment

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

what is this file for?

Copy link
Contributor

Choose a reason for hiding this comment

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

is this the same as the e2e one?

Copy link
Contributor

Choose a reason for hiding this comment

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

What is this file for?

Copy link
Contributor

Choose a reason for hiding this comment

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

are those copied from the e2e?

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this file required ?

@ryanzhang-oss ryanzhang-oss merged commit 7c255d0 into Azure:main Mar 4, 2025
16 checks passed
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.

3 participants