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

feat: warn users of diff architecture image running #1649

Merged
merged 1 commit into from Oct 31, 2023
Merged

Conversation

h4ck3rk3y
Copy link
Contributor

@h4ck3rk3y h4ck3rk3y commented Oct 30, 2023

Description:

We now warn users in the beginning if the image is different from whats expected

Is this change user facing?

YES/NO

References (if applicable):

@h4ck3rk3y h4ck3rk3y marked this pull request as draft October 30, 2023 14:23
@h4ck3rk3y h4ck3rk3y marked this pull request as ready for review October 31, 2023 10:54
@h4ck3rk3y h4ck3rk3y linked an issue Oct 31, 2023 that may be closed by this pull request
@h4ck3rk3y
Copy link
Contributor Author

h4ck3rk3y commented Oct 31, 2023

Screenshot 2023-10-31 at 10 53 41 AM

Its basically this but I don't log the architecture in the initial blue message any more - that was just for test

@cloudflare-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 185561f
Status: ✅  Deploy successful!
Preview URL: https://a1dc7e41.kurtosis-docs.pages.dev
Branch Preview URL: https://gyani-platform.kurtosis-docs.pages.dev

View logs

@h4ck3rk3y h4ck3rk3y added this pull request to the merge queue Oct 31, 2023
Merged via the queue into main with commit 77f2f69 Oct 31, 2023
33 checks passed
@h4ck3rk3y h4ck3rk3y deleted the gyani/platform branch October 31, 2023 13:33
github-merge-queue bot pushed a commit that referenced this pull request Oct 31, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.85.1](0.85.0...0.85.1)
(2023-10-31)


### Features

* initial new enclave manager
([#1603](#1603))
([9944658](9944658))
* warn users of diff architecture image running
([#1649](#1649))
([77f2f69](77f2f69))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
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.

issue a warning if an image of a different platform is running
2 participants