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

[Innovium] Innovium build changes for master branch #13512

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

Keshavg-marvell
Copy link
Contributor

Signed-off-by: Keshav Gupta keshavg@marvell.com

Why I did it
To Fix innovium build issue

Fixing Innovium build for master branch

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 25, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Keshavg-marvell / name: Keshav Gupta (f87a3a9f88ebba72d9af5c5f02d3a691f4a35217)

@lguohan
Copy link
Collaborator

lguohan commented Feb 23, 2023

the build still failed, please check?

Signed-off-by: Keshav Gupta <keshavg@marvell.com>

retaining docker username and tag

rearranging the change to place it at right place

Signed-off-by: Keshav Gupta <keshavg@marvell.com>
@Keshavg-marvell
Copy link
Contributor Author

Keshavg-marvell commented Mar 1, 2023

the build still failed, please check?

Thanks Guohan Lu for checking. Previously it might have some intermittent issue and resulted into build failure . To retrigger the build, I did one cosmetic change and build is fine now. Can you please review the changes?

@rck-innovium
Copy link
Contributor

@lguohan  please review.

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