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.

Sign up for GitHub

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

Jump to bottom

[Release][C#] Source verification for C# on 15.0.0 Release Candidate fails with file does not exist: artifacts/Apache.Arrow/Release/netstandard1.3/Apache.Arrow.pdb #39598

Closed
raulcd opened this issue Jan 14, 2024 · 4 comments · Fixed by #39605
Closed

[Release][C#] Source verification for C# on 15.0.0 Release Candidate fails with file does not exist: artifacts/Apache.Arrow/Release/netstandard1.3/Apache.Arrow.pdb #39598

raulcd opened this issue Jan 14, 2024 · 4 comments · Fixed by #39605
Assignees
@CurtHagenlocher
Labels
Component: C# Component: Release Priority: Blocker Marks a blocker for the release Type: bug
Milestone
15.0.0

Comments

@raulcd
Copy link
Member

raulcd commented Jan 14, 2024

Describe the bug, including details regarding any error messages, version, and platform.

I have been executing the source verification jobs both on the nightly jobs and the maintenance branch and I haven't found any prior issues but as soon as I've created the Release Candidate 0 for 15.0.0 all the C# verification jobs for the sources fail with the following error:

  Successfully created package '/tmp/arrow-15.0.0.NZGxf/apache-arrow-15.0.0/csharp//artifacts/Apache.Arrow.Flight.Sql/Release/Apache.Arrow.Flight.Sql.15.0.0.snupkg'.
file does not exist: artifacts/Apache.Arrow/Release/netstandard1.3/Apache.Arrow.pdb
Failed to verify release candidate. See /tmp/arrow-15.0.0.NZGxf for details.

The Release Candidate 0 jobs that are failing can be seen here:
#39597 (comment)
and one example of failed job here: https://github.com/ursacomputing/crossbow/actions/runs/7519893060/job/20468954451

The maintenance branch successful jobs can be seen here:
#39545 (comment)

Component(s)

C#, Release
@raulcd raulcd added Type: bug Priority: Blocker Marks a blocker for the release labels Jan 14, 2024
@raulcd raulcd added this to the 15.0.0 milestone Jan 14, 2024
@raulcd
Copy link
Member Author

raulcd commented Jan 14, 2024

@CurtHagenlocher @eerhardt any idea what might be going wrong on the C# verification?

@CurtHagenlocher
Copy link
Contributor

CurtHagenlocher commented Jan 14, 2024 via email •
edited
Loading

@raulcd
Copy link
Member Author

raulcd commented Jan 14, 2024

Thanks @CurtHagenlocher no worries. I'll try to investigate tomorrow too if I can see what is going on but knowing that you removed support for 1.3 is a huge pointer to where to investigate. Thank you very much!

@github-actions github-actions bot mentioned this issue Jan 15, 2024
Merged
@raulcd raulcd mentioned this issue Jan 16, 2024
Open
@raulcd
Copy link
Member Author

raulcd commented Jan 16, 2024 •
edited
Loading

@kou I plan to merge this. I have created #39631 to follow up the local build issue. Let me know if you are ok with merging it as is to add to RC1

I meant to post on the PR not the issue :)

CurtHagenlocher added a commit that referenced this issue Jan 16, 2024
@CurtHagenlocher
GH-39598: [C#] Fix verification script (#39605)
d6a8305 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: #39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
@CurtHagenlocher CurtHagenlocher modified the milestones: 15.0.0, 16.0.0 Jan 16, 2024
@raulcd raulcd modified the milestones: 16.0.0, 15.0.0 Jan 16, 2024
raulcd pushed a commit that referenced this issue Jan 16, 2024
@CurtHagenlocher @raulcd
GH-39598: [C#] Fix verification script (#39605)
d82a5d0 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: #39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
idailylife pushed a commit to idailylife/arrow that referenced this issue Jan 18, 2024
@CurtHagenlocher @idailylife
apacheGH-39598: [C#] Fix verification script (apache#39605)
e0bac8a 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: apache#39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
clayburn pushed a commit to clayburn/arrow that referenced this issue Jan 23, 2024
@CurtHagenlocher @clayburn
apacheGH-39598: [C#] Fix verification script (apache#39605)
18fea6d 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: apache#39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@CurtHagenlocher @dgreiss
apacheGH-39598: [C#] Fix verification script (apache#39605)
761decb 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: apache#39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
@CurtHagenlocher @zanmato1984
apacheGH-39598: [C#] Fix verification script (apache#39605)
5153375 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: apache#39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
@CurtHagenlocher @thisisnic
apacheGH-39598: [C#] Fix verification script (apache#39605)
cb522fb 
### What changes are included in this PR?

The verification script is modified to look for the versions of .NET now supported by the package.

### Are these changes tested?

Manually tested the verification command.

* Closes: apache#39598

Authored-by: Curt Hagenlocher <[email protected]>
Signed-off-by: Curt Hagenlocher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CurtHagenlocher CurtHagenlocher

Labels
Component: C# Component: Release Priority: Blocker Marks a blocker for the release Type: bug
Projects
None yet
Milestone
15.0.0
Development

Successfully merging a pull request may close this issue.

GH-39598: [C#] Fix verification script CurtHagenlocher/arrow
2 participants
@raulcd @CurtHagenlocher