Chassis / memcache

A Chassis extension to install and configure memcached on your server.
1 stars 5 forks source link

This extension seems to be broken #21

Closed roborourke closed 6 years ago

roborourke commented 6 years ago
  1. What operating system do you use? OSX

  2. What version of Vagrant are you running? 2.0.0

  3. Are you using VirtualBox or VMWare and which version are you using? Virtualbox 5.1.28

  4. Do you have a custom YAML file? If so, what does it contain? (Please paste in your chassis.local.yaml file)

php: 7.1
multisite: Yes
extensions:
  - chassis/chassis-elasticsearch
  - chassis/memcache
  - javorszky/chassis-openssl
  - chassis/cavalcade
  1. Do you have any Chassis other extensions installed? If so, what are they?

See above.


I get the following error:

Error: Puppet::Parser::AST::Resource failed with error ArgumentError: Could not find declared class memcached at /tmp/vagrant-puppet/manifests-846018e2aa141a5eb79a64b4015fc5f3/development.pp:12 on node vagrant.local
==> default: Error: Puppet::Parser::AST::Resource failed with error ArgumentError: Could not find declared class memcached at /tmp/vagrant-puppet/manifests-846018e2aa141a5eb79a64b4015fc5f3/development.pp:12 on node vagrant.local
roborourke commented 6 years ago

Missing chassis.pp file...

rmccue commented 6 years ago

You may need to update your copy of Chassis, as the extension requires support for v2.

BronsonQuick commented 6 years ago

@rmccue Yeah I figured it was that: https://github.com/Chassis/memcache/pull/16#issuecomment-373168693 Should I go through all the extensions and add back chassis.pp for all the people who don't keep Chassis up to date?

BronsonQuick commented 6 years ago

Alrighty. So we experimented trying to use chassis.pp ti run a notice for people to update Chassis core if they were behind but we didn't have any success. When we get issues like this we'll just have to tell people to update Chassis until we ship https://github.com/Chassis/Chassis/pull/504

roborourke commented 6 years ago

@rmccue @BronsonQuick this was with a fresh clone of chassis master and the extension so maybe something's up with the v2 extension logic?

rmccue commented 6 years ago

Reading your error message again:

Error: Puppet::Parser::AST::Resource failed with error ArgumentError: Could not find declared class memcached

Did you clone this down as memcached instead of memcache?

roborourke commented 6 years ago

That shouldn't matter but I tried with both variations and got the same error. Adding the chassis.pp was the only thing that fixed it.

rmccue commented 6 years ago

Due to the way that extensions work, it does matter what the directory is called. I’ll investigate further tomorrow.

rmccue commented 6 years ago

I tested this out and it works perfectly fine with latest Chassis and memcache extension. The only way I could replicate the original error was by cloning the extension to a memcached directory instead; as mentioned above, the directory name has to match exactly.

Reverting the chassis.pp addition.