Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

Update infrastructure #68

Merged
merged 59 commits into from
Oct 30, 2022
Merged

Update infrastructure #68

merged 59 commits into from
Oct 30, 2022

Conversation

fasmat
Copy link
Member

@fasmat fasmat commented Oct 27, 2022

This fixes the build pipelines for gpu-post:

  • Update zlib from v1.2.11 to v1.2.13
  • Update build pipelines:
    • Use Ubuntu 22.04 instead of 20.04
    • Use Windows Server 2022 instead of 2016
    • Update Vulkan on Ubuntu from 1.2.154 to 1.3.231
    • Update CUDA on Ubuntu from 11.4 to 11.8 and fix installation script
    • Update Vulkan on macOS from 1.2.170.0 to 1.3.231.0 and fix installation script
    • Update CUDA on Windows from 11.2.2.46133 to 11.8.0.52206
    • Update Vulkan on windows from 1.2.154.1 to 1.2.182.0

@fasmat fasmat requested a review from pigmej October 27, 2022 13:00
@fasmat fasmat self-assigned this Oct 27, 2022
Copy link
Member

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

I checked again, and currently, Running tests fail with Vulkan SDK 1.3.231.1 on self-hosted mac m1.
It passes on 1.2.198.1

I'd, for now, downgrade to 1.2.x everywhere.

Besides that all good IMHO.

@fasmat fasmat requested a review from pigmej October 27, 2022 15:17
Copy link
Member

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

The last remaining question would be why we have to use 1.2.198.1 on m1 while all others work with 1.3.x

But given that somehow m1 seems to be also super slow, (labels-count takes 12+minutes while other platforms <10m) then maybe the problem is "somewhere" else probably.

Copy link
Member

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

These are few nit picks, we can do them here or in separate :)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@fasmat fasmat merged commit be4f560 into develop Oct 30, 2022
@fasmat fasmat deleted the update-infrastructure branch October 30, 2022 17:59
@fasmat fasmat mentioned this pull request Nov 2, 2022
@fasmat fasmat linked an issue Nov 2, 2022 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix CI for gpu-post
2 participants