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

[TESTING ONLY] #2063

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

matejvasek
Copy link
Contributor

No description provided.

Signed-off-by: Matej Vašek <mvasek@redhat.com>
@matejvasek matejvasek requested review from a team as code owners February 14, 2024 04:43
@github-actions github-actions bot added this to the 0.34.0 milestone Feb 14, 2024
@github-actions github-actions bot added type/enhancement Issue that requests a new feature or improvement. type/chore Issue that requests non-user facing changes. labels Feb 14, 2024
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

Attention: Patch coverage is 85.71429% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 79.72%. Comparing base (6fc092b) to head (d593b0d).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2063      +/-   ##
==========================================
- Coverage   79.72%   79.72%   -0.00%     
==========================================
  Files         176      176              
  Lines       13263    13277      +14     
==========================================
+ Hits        10573    10584      +11     
- Misses       2021     2022       +1     
- Partials      669      671       +2     
Flag Coverage Δ
os_linux 78.65% <85.72%> (-<0.01%) ⬇️
os_macos 76.35% <76.20%> (+0.01%) ⬆️
os_windows 79.11% <85.72%> (-<0.01%) ⬇️
unit 79.72% <85.72%> (-<0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@github-actions github-actions bot removed the type/chore Issue that requests non-user facing changes. label Apr 4, 2024
@jjbustamante
Copy link
Member

I created simple acceptance test with this code, and @matejvasek you were right I got the following error:

  Host Settings:
          Binds: 'pack-cache-some-org_gbmqdqekqf_latest-bc166514d2fd.build:/cache pack-layers-smuiylvmuq:/layers pack-app-ohcxidehgd:/workspace'
          Network Mode: 'host'
        ERROR: failed to build: executing lifecycle: failed to create 'creator' container: "specify mac-address per network" requires API version 1.44, but the Docker daemon API version is 1.38

We are actually using Docker API version I will try to update and push the acceptance test to this branch

…r API 1.44

Signed-off-by: Juan Bustamante <jbustamante@vmware.com>
@github-actions github-actions bot added the type/chore Issue that requests non-user facing changes. label Apr 4, 2024
Signed-off-by: Juan Bustamante <jbustamante@vmware.com>
@jjbustamante
Copy link
Member

Damn, the linux runner is using Docker Server 24.0.9, and this version only support version 1.43

Signed-off-by: Juan Bustamante <jbustamante@vmware.com>
@jjbustamante
Copy link
Member

jjbustamante commented Apr 4, 2024

This table shows the Docker version used by pack and the runners, we need to move to 25.0 in order to support setting the mac-address but, the ubuntu latest runner doesn't support that version yet, also, we will need to update the self hosted windows runner

Docker version Maximum API version Runner Version Note
26.0 1.45
25.0 1.44 Expected version
24.0 1.43 Linux runner
23.0 1.42
20.10 1.41 Windows runner <- I upgraded to this in this PR
19.03 1.40
18.09 1.39
18.06 1.38 current pack version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants