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

bump crib-deploy-environment to latest #15766

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

rafaelfelix
Copy link
Collaborator

@rafaelfelix rafaelfelix commented Dec 19, 2024

Required to fix crib-integration-test workflow

@rafaelfelix rafaelfelix requested review from a team as code owners December 19, 2024 10:01
Copy link
Contributor

github-actions bot commented Dec 19, 2024

AER Report: CI Core

aer_workflow , commit , Detect Changes , Scheduled Run Frequency , Clean Go Tidy & Generate , Flakeguard Root Project / Get Tests To Run , GolangCI Lint , Core Tests (go_core_tests) , test-scripts , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , Flakeguard Deployment Project , Flakey Test Detection , Flakeguard Root Project / Run Tests , Flakeguard Root Project / Report , lint , SonarQube Scan

1. Permission denied for golangci-lint report file:[Golang Lint]

Source of Error:
Golang Lint ()	2024-12-19T11:24:51.8180128Z level=error msg="Running error: can't create output for golangci-lint-report.xml: open golangci-lint-report.xml: permission denied"
**Why**: The `golangci-lint` action attempted to create or write to the `golangci-lint-report.xml` file but encountered a permission issue, likely due to insufficient write permissions in the specified directory.

Suggested fix: Ensure that the directory where golangci-lint-report.xml is being created has the appropriate write permissions. You can add a step to change the directory permissions before running the golangci-lint action.

2. Directory prefix does not contain main module:[Golang Lint]

Source of Error:
Golang Lint ()	2024-12-19T11:24:51.8178453Z level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix / does not contain main module or its selected dependencies"
**Why**: The `golangci-lint` action is running in the root directory (`/`), which does not contain the Go module or its dependencies, causing a type-checking error.

Suggested fix: Set the correct working directory for the golangci-lint action to the directory containing the Go module (e.g., working-directory: ./path/to/module).

3. No such file or directory for golangci-lint report:[Golang Lint]

Source of Error:
Golang Lint ()	2024-12-19T11:24:51.8574351Z cat: .//golangci-lint-report.xml: No such file or directory
**Why**: The `golangci-lint-report.xml` file was not created due to the previous permission error, leading to a "No such file or directory" error when attempting to read the file.

Suggested fix: Resolve the permission issue first to ensure the file is created. Then, verify the file path and existence before attempting to read it.

4. No files found for upload:[Golang Lint]

Source of Error:
Golang Lint ()	2024-12-19T11:24:52.1450554Z ##[warning]No files were found with the provided path: .//golangci-lint-report.xml. No artifacts will be uploaded.
**Why**: The `golangci-lint-report.xml` file was not found because it was not created due to the permission error.

Suggested fix: Ensure the file is created by fixing the permission issue. Verify the file path and existence before attempting to upload it.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@rafaelfelix rafaelfelix force-pushed the fix-crib-integration-test branch from 6d5aa56 to b2197bf Compare December 19, 2024 11:23
@scheibinger scheibinger added this pull request to the merge queue Dec 19, 2024
Merged via the queue into develop with commit 8a87d1d Dec 19, 2024
101 of 102 checks passed
@scheibinger scheibinger deleted the fix-crib-integration-test branch December 19, 2024 12:25
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