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

Boostrap script on 64-bit Pi 3 does not detect architecture #905

Closed
meggiebot opened this issue Jul 5, 2016 · 8 comments
Closed

Boostrap script on 64-bit Pi 3 does not detect architecture #905

meggiebot opened this issue Jul 5, 2016 · 8 comments
Assignees
Labels

Comments

@meggiebot
Copy link

meggiebot commented Jul 5, 2016

Description of Issue/Question

On 64-bit Pi3 the architecture wasn't detected.

Reported via Twitter: https://twitter.com/joesusecom/status/748867271049809921

@joesusecom
Copy link

joesusecom commented Jul 5, 2016

I've tried on the Raspberry Pi Model B (32-bit). Doesn't work there either, unless I'm checking out of git. The error message is:

bootstrap_salt.sh: 2822: bootstrap_salt.sh: repo_arch: parameter not set

I'll try the latest script from

wget https://bootstrap.saltstack.com/develop

Seems that
https://docs.saltstack.com/en/latest/topics/tutorials/salt_bootstrap.html

has an error:

If I do what it says in "Downloading the script from develop branch:"

wget https://bootstrap.saltstack.com/develop
sudo sh bootstrap_salt.sh

that won't work, because the script is actually saved as "develop".

@rallytime
Copy link
Contributor

@joesusecom I suspect that we'll need a change similar to #899, only we'll need to also check for the armv8 arch cpu in addition to checking for armv7l.

@joesusecom
Copy link

joesusecom commented Jul 6, 2016

Actually the check for armv7l (which was added after I tested the script) should work with the Raspberry Pi 3. It reports as armv7l. For the Pi B we'd need armv6l. I verified with the same SD card that the same Raspberry image also runs on the Pi B.

@rallytime
Copy link
Contributor

@joesusecom Oh, that is good to know about the armv7l reporting on Pi 3. I had read some issues online saying that is reported, even though the Pi 3 wiki says it should be armv8.

that won't work, because the script is actually saved as "develop".

Can you clarify what you mean here? You should be able to run the script with whatever name it downloads as.

@joesusecom
Copy link

Am 06.07.2016 um 17:40 schrieb Nicole Thomas <notifications@github.commailto:notifications@github.com>:

@joesusecomhttps://github.com/joesusecom Oh, that is good to know about the armv7l reporting on Pi 3. I had read some issues online saying that is reported, even though the Pi 3 wiki says it should be armv8.

that won't work, because the script is actually saved as "develop".

Can you clarify what you mean here? You should be able to run the script with whatever name it downloads as.

Sure, but the docs at

https://docs.saltstack.com/en/latest/topics/tutorials/salt_bootstrap.html

say

wget https://bootstrap.saltstack.com/develop

sudo sh bootstrap_salt.sh

If you do exactly that, it can't work.

Also, many of the bash examples on that page are marked up as Python ...

Cheers
Joachim


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//issues/905#issuecomment-230811843, or mute the threadhttps://github.com/notifications/unsubscribe/ADmXg-aDcmHnVb1G1-VB_GodDu1J0ChUks5qS8x8gaJpZM4JFfft.

@rallytime
Copy link
Contributor

Heh. I actually haven't seen those docs before.

You should be able to wget -O bootstrap_salt.sh https://bootstrap.saltstack.com/develop and then sh bootstrap_salt.sh.

@rallytime
Copy link
Contributor

@joesusecom I have fixed those instructions in the above pull request. Thanks for letting me know.

@rallytime
Copy link
Contributor

@joesusecom I just realized that the link you were talking about doesn't actually pull down the correct develop version. So even if you were running what you thought was the develop version, it was still the latest stable.

We're releasing a new stable version of the bootstrap script today, so you should be able to try this again from stable soon.

I spoke off-line with @Ch3LL, who did some testing on our Raspberry Pis, (she tested both Pi 2 and Pi 3) and the new stable version works correctly. Since we've confirmed this on our end, I am going to close this, but if you run into any trouble with the new stable script, please let me know and we'll take another look. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants