Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Add CAPI metadata.yaml to the e2e #260

Merged

Conversation

namnx228
Copy link
Member

@namnx228 namnx228 commented Jul 28, 2021

This PR adds CAPI metadata.yaml to the e2e folder, so the problem with clusterctl init can be solved. This metadata.yaml differs from the one in the root directory. It documents the release series of cluster-api provider, so it is the file located in the root directory of cluster-api repo.

failed to run clusterctl init
    Unexpected error:
        <*errors.withStack | 0xc001656870>: {
            error: <*errors.withMessage | 0xc0008fb2e0>{
                cause: <*errors.withStack | 0xc001656840>{
                    error: <*errors.withMessage | 0xc0008fb2c0>{
                        cause: <*fs.PathError | 0xc000b8a000>{
                            Op: "stat",
                            Path: "/home/****/tested_repo/_artifacts/repository/cluster-api/v0.4.0/metadata.yaml",
                            Err: <syscall.Errno>0x2,
                        },
                        msg: "failed to read file \"/home/****/tested_repo/_artifacts/repository/cluster-api/v0.4.0/metadata.yaml\" from local release v0.4.0",
                    },
                    stack: [0x193fcc5, 0x193c7d6, 0x194d904, 0x194d1be, 0x1976742, 0x197df55, 0x1a00bfc, 0x1a05aa8, 0x78b123, 0x78ad3c, 0x78a307, 0x790e2f, 0x7904d2, 0x7b90b1, 0x7b8bc7, 0x7b83b7, 0x7baac6, 0x7c8178, 0x7c7e27, 0x19ff3b8, 0x527c2f, 0x4744e1],
                },
                msg: "failed to read \"metadata.yaml\" from the repository for provider \"cluster-api\"",
            },
            stack: [0x193c91d, 0x194d904, 0x194d1be, 0x1976742, 0x197df55, 0x1a00bfc, 0x1a05aa8, 0x78b123, 0x78ad3c, 0x78a307, 0x790e2f, 0x7904d2, 0x7b90b1, 0x7b8bc7, 0x7b83b7, 0x7baac6, 0x7c8178, 0x7c7e27, 0x19ff3b8, 0x527c2f, 0x4744e1],
        }
        failed to read "metadata.yaml" from the repository for provider "cluster-api": failed to read file "/home/****/tested_repo/_artifacts/repository/cluster-api/v0.4.0/metadata.yaml" from local release v0.4.0: stat /home/****/tested_repo/_artifacts/repository/cluster-api/v0.4.0/metadata.yaml: no such file or directory
    occurred

@metal3-io-bot metal3-io-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 28, 2021
@namnx228
Copy link
Member Author

/test-integration
/test-v1a5-e2e

@namnx228
Copy link
Member Author

/assign @furkatgofurov7
/cc @wgslr
/cc @fmuyassarov

@namnx228 namnx228 force-pushed the add-CAPI-metadata-yaml-e2e-nam branch from 1d8f40f to 47e62ef Compare July 28, 2021 13:45
@metal3-io-bot metal3-io-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 28, 2021
@namnx228
Copy link
Member Author

/test-integration
/test-v1a5-e2e

@namnx228 namnx228 force-pushed the add-CAPI-metadata-yaml-e2e-nam branch from 47e62ef to 8b2a806 Compare July 29, 2021 07:32
@namnx228
Copy link
Member Author

/hold

@metal3-io-bot metal3-io-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 29, 2021
@namnx228
Copy link
Member Author

/test-v1a5-e2e

@namnx228 namnx228 force-pushed the add-CAPI-metadata-yaml-e2e-nam branch from 8b2a806 to b2baa55 Compare July 29, 2021 11:53
@namnx228 namnx228 force-pushed the add-CAPI-metadata-yaml-e2e-nam branch from b2baa55 to 1349f23 Compare July 29, 2021 11:53
@wgslr
Copy link
Member

wgslr commented Jul 29, 2021

/approve

@namnx228
Copy link
Member Author

/hold cancel

@metal3-io-bot metal3-io-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 30, 2021
@namnx228
Copy link
Member Author

/test-integration
/test-v1a5-e2e

Copy link
Member

@furkatgofurov7 furkatgofurov7 left a comment

Choose a reason for hiding this comment

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

/approve

@metal3-io-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: furkatgofurov7, wgslr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 30, 2021
@Rozzii
Copy link
Member

Rozzii commented Aug 2, 2021

/lgtm

@metal3-io-bot
Copy link
Contributor

@Rozzii: adding LGTM is restricted to approvers and reviewers in OWNERS files.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link
Member

@fmuyassarov fmuyassarov left a comment

Choose a reason for hiding this comment

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

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Aug 2, 2021
@metal3-io-bot metal3-io-bot merged commit 4f56d61 into metal3-io:master Aug 2, 2021
@kashifest kashifest deleted the add-CAPI-metadata-yaml-e2e-nam branch August 23, 2022 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants