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

Zarf prints incorrect architecture information in error message on package deploy #1493

Closed
lucasrod16 opened this issue Mar 28, 2023 · 0 comments · Fixed by #1494
Closed
Assignees
Labels
bug 🐞 Something isn't working

Comments

@lucasrod16
Copy link
Contributor

lucasrod16 commented Mar 28, 2023

Environment

Device and OS: M1 Mac
App version: v0.25.0
Kubernetes distro being used: k3d (k3s in docker)
Other: This is agnostic to device/OS and k8s distro. The information about package and cluster architecture just need to be swapped when an application package's architecture doesn't match a cluster's architecture.

Steps to reproduce

  1. Create a cluster. In this case, a k3d cluster on an M1 Mac:
k3d cluster create

The architecture of the nodes is arm64.

  1. Initialize the cluster with an arm64 init package:
zarf init
  1. Attempt to deploy a zarf package with amd64 architecture to the arm64 cluster:
zarf package deploy zarf-package-cool-app-amd64

Expected result

The deployment fails because the architectures do not match, and zarf tells me why.

Actual Result

The deployment fails because the architectures do not match, and zarf tells me why, but prints out backwards/incorrect architecture data in the error message.

Visual Proof (screenshots, videos, text, etc)

Screenshot 2023-03-27 at 8 08 16 PM

Severity/Priority

Unsure.

@lucasrod16 lucasrod16 added the bug 🐞 Something isn't working label Mar 28, 2023
@github-project-automation github-project-automation bot moved this to New Requests in Zarf Project Board Mar 28, 2023
@lucasrod16 lucasrod16 self-assigned this Mar 28, 2023
@github-project-automation github-project-automation bot moved this from New Requests to Done in Zarf Project Board Mar 28, 2023
Racer159 pushed a commit that referenced this issue Mar 28, 2023
…es don't match (#1494)

## Description
Fix package deploy error message when package and cluster architectures
don't match

## Related Issue

Fixes #1493

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)

## Checklist before merging

- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow)
followed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant