Parallels/Arm tweaks #941

tomjn · 2021-12-26T14:36:04Z

Fixes some issues on M1 Macs and Parallels

BronsonQuick

Thanks so much for making a start on this! Most appreciated. I've left a few comments to follow up on as I think we might be able to simplify it 🤞🏻

Vagrantfile

puppet/modules/chassis/lib/puppet/parser/functions/sz_extensions.rb

tomjn · 2021-12-26T22:29:17Z

It might be related to using Puppet 7

…

On Sun, 26 Dec 2021 at 22:28, Bronson Quick ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In puppet/modules/chassis/lib/puppet/parser/functions/sz_extensions.rb <#941 (comment)>: > @@ -1,15 +1,15 @@ require "/vagrant/puppet/chassis" module Puppet::Parser::Functions + newfunction(:chassis_get_global_extensions, :type => :rvalue) do |args| That's a weird one! No worries. It won't affect anything for other providers so I'm happy to leave this then! — Reply to this email directly, view it on GitHub <#941 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAOLZ52P6IC2ZWYCCUDRHLUS6JILANCNFSM5KY7DRSQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you authored the thread.Message ID: ***@***.***>

…iginal

puppet/preprovision.sh

tomjn · 2021-12-26T22:49:41Z

current result of this PR on Parallels/Arm64:

==> default: Checking if box 'mpasternak/focal64-arm' version '202110.0.0' is up to date...
==> default: Running provisioner: shell...
    default: Running: /var/folders/dw/wdtnsdn154db04ymxv90t9tm0000gn/T/vagrant-shell20211226-64180-1554jem.sh
==> default: Running provisioner: shell...
    default: Running: inline script
    default: Warning: ModuleLoader: module 'elasticsearch' has unresolved dependencies - it will only see those that are resolved. Use 'puppet module list --tree' to see information about modules\n   (file & line not available)
    default: Error: Could not find resource 'Exec[systemctl-daemon-reload]' in parameter 'notify' (file: /vagrant/extensions/cavalcade/modules/cavalcade/manifests/init.pp, line: 47) on node siemens.local
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

tomjn · 2021-12-26T22:50:08Z

That's running on Puppet 5

$ puppet --version
5.5.10

BronsonQuick · 2021-12-26T22:57:18Z

Ahh right. I'm wonder if the submodules update I merged in from you in ES caused that. Can you please try changing the ES extension to be a commit before the submodule update and see if that works? If that's the case then I'll need to revert that.

I'll try a fresh provision of Chassis just with the ES extension now as well.

tomjn · 2021-12-26T23:04:03Z

Can you please try changing the ES extension to be a commit before the submodule update and see if that works?

Nope because then it'll fail complaining Ubuntu 20 isn't supported in the Java puppet config

BronsonQuick · 2021-12-26T23:06:06Z

No worries. I just did a fresh provision with ES and didn't have any issues so that's all good as you mentioned.

I did notice:

default: Error: Could not find resource 'Exec[systemctl-daemon-reload]' in parameter 'notify' (file: /vagrant/extensions/cavalcade/modules/cavalcade/manifests/init.pp, line: 47) on node siemens.local

I didn't realise Cavalcade was using that Exec function so I'll open a new issue for adding that back and I'll fix that now.

tomjn · 2021-12-26T23:52:26Z

@BronsonQuick that helped, it progresses to this point now:

ontent as '{md5}b80e6a642bdcca742ed1c4f28e6861d3'
    default: Notice: /Stage[main]/Chassis::Php/Package[php7.4-curl]/ensure: created
    default: Notice: /Stage[main]/Chassis::Php/Package[php7.4-gd]/ensure: created
    default: Notice: /Stage[main]/Chassis::Php/Package[php7.4-mysql]/ensure: created
    default: Notice: /Stage[main]/Chassis/Package[nginx]/ensure: created
    default: Notice: /Stage[main]/Chassis/Package[sendmail]/ensure: created
    default: Notice: /Stage[main]/Chassis/Exec[systemctl-daemon-reload]: Triggered 'refresh' from 1 event
    default: Notice: /Stage[main]/Chassis/Exec[restart-avahi]/returns: executed successfully
    default: Notice: /Stage[main]/Chassis_openssl/File[/etc/nginx/sites-available/siemens.local.d]/ensure: created
    default: Notice: /Stage[main]/Chassis_openssl/File[/etc/nginx/sites-available/siemens.local.d/ssl]/ensure: defined content as '{md5}d43aa1e332fe578d2be4a6a95237f5c9'
    default: Notice: /Stage[main]/Chassis/File[/etc/nginx/nginx.conf]/content: content changed '{md5}d419ab8777e5d98481e17896e260dfe6' to '{md5}8e234d2d02906f318676d26b75f49c42'
    default: Notice: /Stage[main]/Chassis::Hosts/Service[chassis-hosts]/ensure: ensure changed 'stopped' to 'running'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/vagrant/index.php]/ensure: defined content as '{md5}60744b085e3f687928a2eb1177e62b42'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/vagrant/local-config-db.php]/ensure: defined content as '{md5}2ad40bdacccd17abd68b16f7bb5a041e'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/vagrant/local-config-extensions.php]/ensure: defined content as '{md5}9881805b872b4ef952b6ff9db3098457'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/home/vagrant/.wp-cli]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/home/vagrant/.wp-cli/config.yml]/ensure: defined content as '{md5}beac827deb496c6edf1f38124a029ac2'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/File[/vagrant/wp-cli.yml]/ensure: defined content as '{md5}ee75a80af6dcfe7712a0477afc35f690'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/File[/chassis/wordpress]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/File[/etc/nginx/sites-available/siemens.local]/ensure: defined content as '{md5}3e59756e96b77e188aacfabcf689bb59'
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/File[/etc/nginx/sites-enabled/siemens.local]/ensure: created
    default: Notice: /Stage[main]/Chassis/Service[nginx]: Triggered 'refresh' from 4 events
    default: Notice: /Stage[main]/Main/File[/var/www/html/index.html]/ensure: defined content as '{md5}364abbbe228cf1a9f6beea99ce4f8e45'
    default: Notice: /Stage[main]/Mysql::Client::Install/Package[mysql_client]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Mysql::Db[wordpress]/Mysql_database[wordpress]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Mysql::Db[wordpress]/Mysql_user[wordpress@localhost]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Mysql::Db[wordpress]/Mysql_grant[wordpress@localhost/wordpress.*]/ensure: created
    default: Notice: /Stage[main]/Wp::Cli/File[/usr/local/src/wp-cli]/ensure: created
    default: Notice: /Stage[main]/Wp::Cli/File[/usr/local/src/wp-cli/bin]/ensure: created
    default: Notice: /Stage[main]/Wp::Cli/Exec[wp-cli download]/returns: executed successfully
    default: Notice: /Stage[main]/Wp::Cli/File[/usr/local/src/wp-cli/bin/wp]/mode: mode changed '0644' to '0755' (a+x)
    default: Notice: /Stage[main]/Wp::Cli/File[/usr/bin/wp]/ensure: created
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Wp::Site[/chassis/wordpress]/Exec[wp install /chassis/wordpress]/returns: Error: This does not seem to be a WordPress installation.
    default: Notice: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Wp::Site[/chassis/wordpress]/Exec[wp install /chassis/wordpress]/returns: Pass --path=`path/to/wordpress` or run `wp core download`.
    default: Error: '/usr/bin/wp core install --url='http://siemens.local/' --title='Chassis Site' --admin_email='admin@example.com' --admin_name='admin' --admin_password='password'' returned 1 instead of one of [0]
    default: Error: /Stage[main]/Main/Chassis::Wp[siemens.local]/Chassis::Site[siemens.local]/Wp::Site[/chassis/wordpress]/Exec[wp install /chassis/wordpress]/returns: change from 'notrun' to ['0'] failed: '/usr/bin/wp core install --url='http://siemens.local/' --title='Chassis Site' --admin_email='admin@example.com' --admin_name='admin' --admin_password='password'' returned 1 instead of one of [0]
    default: Notice: Applied catalog in 278.96 seconds
==> default: Running provisioner: shell...
    default: Running: /var/folders/dw/wdtnsdn154db04ymxv90t9tm0000gn/T/vagrant-shell20211226-67398-bte9e2.sh
==> default: Running provisioner: set_constants (shell)...
    default: Running: /var/folders/dw/wdtnsdn154db04ymxv90t9tm0000gn/T/vagrant-shell20211226-67398-9f8a9t.sh
==> default: Running action triggers after up ...
==> default: Running trigger...

chassis on  arm-tweaks via 🐘 v8.1.1 via ⍱ v2.2.19 took 4m53s 
❯

tomjn · 2021-12-26T23:52:58Z

this is my test config btw:

paths:
    base: ..
    content: content
    wp: wordpress

hosts:
    - siemens.local

_mode: base

# Define database prefix, since we have tests as well
database:
    name: wordpress
    user: wordpress
    password: vagrantpassword
    prefix: wp_

# Define the extensions we want
extensions:
    - chassis/xdebug
    - chassis/tester
    - chassis/sequelpro
    - chassis/cavalcade
    - chassis/mailhog
    - chassis/memcache
    - chassis/phpcs
    - chassis/chassis_openssl
    - chassis/redis
    - chassis/chassis_elasticsearch
    - chassis/runkit7

elasticsearch:
  repo_version: '7.x'
  version: '7.9.3'
  host: '0.0.0.0'
  port: 9200
  timeout: 10
  instances:
    - 'es'
  jvm_options:
    - '-Xms256m'
    - '-Xmx256m'

BronsonQuick · 2021-12-26T23:55:36Z

@tomjn That's getting super close! 🕺 One thing to confirm, is your config file in the root directory? Because paths only work if the config is in the root directory.

tomjn · 2021-12-27T00:11:52Z

@BronsonQuick that did the trick, I did another vagrant up after some direnv trickery and http://siemens.local served me the WP install screen

BronsonQuick · 2021-12-27T00:30:48Z

@tomjn That's awesome news! Nice work Sir!

kadamwhite · 2022-01-03T16:06:51Z

I pulled this to test locally on a fresh M1 system, and I'm stalling out due to an unavailable box:

Bringing machine 'default' up with 'parallels' provider...
==> default: Running action triggers before up ...
==> default: Running trigger...
==> default: Box 'chassis/chassis' could not be found. Attempting to find and install...
    default: Box Provider: parallels
    default: Box Version: >= 4.0, < 5.0
==> default: Loading metadata for box 'chassis/chassis'
    default: URL: https://vagrantcloud.com/chassis/chassis
The box you're attempting to add doesn't support the provider
you requested. Please find an alternate box or use an alternate
provider. Double-check your requested provider to verify you didn't
simply misspell it.

If you're adding a box from HashiCorp's Vagrant Cloud, make sure the box is
released.

Name: chassis/chassis
Address: https://vagrantcloud.com/chassis/chassis
Requested provider: [:parallels]
kadam@Ks-MacBook-Pro chassis % git status
On branch arm-tweaks
Your branch is up to date with 'tomjn/arm-tweaks'.

nothing to commit, working tree clean

This was with a fresh checkout of chassis/chassis and no custom config.

kadamwhite

Update: Tom instructed me to try base box mode (which I hadn't known about, thanks!) and with this config.local.yaml

# config.local.yaml

_mode: base

I was able to get to the WP login screen successfully 👍 🚀 ✅ And it runs fast!

I'll defer to @BronsonQuick but this is working for me, thanks for taking this on @tomjn

kadamwhite · 2022-01-03T21:14:00Z

puppet/preprovision.sh

@@ -38,7 +38,6 @@ if [[ ! -f /etc/chassis-updated ]]; then
 	# Update apt
 	sudo apt-get update

-	# Install/Upgrade Puppet


Why remove this line?

Because I am a stupidhead 😂 I probably just missed it when I was adding it back when Puppet v7 didn't play nicely!

Vagrantfile

Co-authored-by: K Adam White <kadamwhite@users.noreply.github.com>

tomjn · 2022-01-31T15:30:22Z

I'm struggling to find the bandwidth to write docs for this, the general gist is:

set the mode to basebox
install Parallels Pro/Business
install vagrant-parallels plugin
vagrant up as usual

Where that goes and how descriptive it needs to be to fit into the Chassis docs? ¯\_(ツ)_/¯

tomjn · 2022-01-31T15:30:56Z

As I understand it, the docs needing to be written are the blocker for this, I'm more than happy for someone to pick that up, I am not a blocker for this

BronsonQuick · 2022-02-01T02:00:11Z

Hey @tomjn, the docs go in the docs folder. Thanks for the notes. I'll pick this up in the next couple of days and add the docs. Thanks again for the PR and troubleshooting. Most appreciated!

BronsonQuick · 2022-02-05T21:54:59Z

I've added a separate issue for me to do the docs: #953 I'll merge this in and type up some docs now. Thanks again @tomjn and @kadamwhite 🙌🏻

tomjn · 2022-02-06T01:46:11Z

@BronsonQuick did you mean to merge this PR? I'm not seeing equivalent commits merged elsewhere

BronsonQuick · 2022-02-06T04:21:12Z

@tomjn I did hit merge this morning and it looks like it didn't go through. I had two other PRs timing out on their checks too so I'm guessing something was up with Github. Thanks for the heads up! Let me try that again!

I've added the docs for this.

tomjn added 3 commits December 26, 2021 14:32

fixes for parallels folder mounts

dcaa316

don't just assume the box has puppet 7 pre-installed

b859e12

swap the order of functions (fixes an issue)

09b24c7

BronsonQuick previously requested changes Dec 26, 2021

View reviewed changes

Vagrantfile Outdated Show resolved Hide resolved

Vagrantfile Show resolved Hide resolved

Vagrantfile Show resolved Hide resolved

puppet/modules/chassis/lib/puppet/parser/functions/sz_extensions.rb Show resolved Hide resolved

BronsonQuick mentioned this pull request Dec 26, 2021

Revert "Remove our installation of puppet" #942

Merged

tomjn added 3 commits December 26, 2021 22:33

Merge branch 'main' into arm-tweaks

b45915d

remove extra puppet install check and add a conditional around the or…

21cd5b5

…iginal

remove puppet install conditional

032474f

BronsonQuick reviewed Dec 26, 2021

View reviewed changes

puppet/preprovision.sh Outdated Show resolved Hide resolved

BronsonQuick mentioned this pull request Dec 26, 2021

Error: Could not find resource 'Exec[systemctl-daemon-reload]' #943

Closed

Merge branch 'main' into arm-tweaks

619b38c

kadamwhite approved these changes Jan 3, 2022

View reviewed changes

Update Vagrantfile

e448540

Co-authored-by: K Adam White <kadamwhite@users.noreply.github.com>

BronsonQuick self-assigned this Feb 1, 2022

BronsonQuick added the Docs label Feb 1, 2022

BronsonQuick added the Enhancement label Feb 1, 2022

BronsonQuick mentioned this pull request Feb 5, 2022

Document Macbook M1 usage #953

Closed

BronsonQuick closed this Feb 5, 2022

BronsonQuick reopened this Feb 6, 2022

BronsonQuick merged commit 49a2c83 into Chassis:main Feb 6, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Parallels/Arm tweaks #941

Parallels/Arm tweaks #941

tomjn commented Dec 26, 2021

BronsonQuick left a comment

tomjn commented Dec 26, 2021 via email

tomjn commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021

tomjn commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021 •

edited

tomjn commented Dec 27, 2021

BronsonQuick commented Dec 27, 2021

kadamwhite commented Jan 3, 2022 •

edited

kadamwhite left a comment

kadamwhite Jan 3, 2022

BronsonQuick Jan 5, 2022

tomjn commented Jan 31, 2022

tomjn commented Jan 31, 2022

BronsonQuick commented Feb 1, 2022

BronsonQuick commented Feb 5, 2022

tomjn commented Feb 6, 2022

BronsonQuick commented Feb 6, 2022

Parallels/Arm tweaks #941

Parallels/Arm tweaks #941

Conversation

tomjn commented Dec 26, 2021

BronsonQuick left a comment

Choose a reason for hiding this comment

tomjn commented Dec 26, 2021 via email

tomjn commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021

tomjn commented Dec 26, 2021

tomjn commented Dec 26, 2021

BronsonQuick commented Dec 26, 2021 • edited

tomjn commented Dec 27, 2021

BronsonQuick commented Dec 27, 2021

kadamwhite commented Jan 3, 2022 • edited

kadamwhite left a comment

Choose a reason for hiding this comment

kadamwhite Jan 3, 2022

Choose a reason for hiding this comment

BronsonQuick Jan 5, 2022

Choose a reason for hiding this comment

tomjn commented Jan 31, 2022

tomjn commented Jan 31, 2022

BronsonQuick commented Feb 1, 2022

BronsonQuick commented Feb 5, 2022

tomjn commented Feb 6, 2022

BronsonQuick commented Feb 6, 2022

BronsonQuick commented Dec 26, 2021 •

edited

kadamwhite commented Jan 3, 2022 •

edited