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

Ubuntu 16.04. compatibility #107

Closed
goatherd opened this issue Jun 28, 2016 · 13 comments · Fixed by #135
Closed

Ubuntu 16.04. compatibility #107

goatherd opened this issue Jun 28, 2016 · 13 comments · Fixed by #135
Assignees
Milestone

Comments

@goatherd
Copy link

Found two issues when setting this up for Ubuntu 16.04.

The packed client dev package is called libmariadb-client-lgpl-dev.
It should be possible to modify the hard-coded libmariadbclient-dev through configuration.

Ubuntu 16.04. comes with native mariadb 10.0 packages.
It will install but there are warnings from the cookbook stating the OS does not have the packages.

@sinfomicien
Copy link
Contributor

Hi goatherd !

Is it possible for you, to prepare a PR for that ? I'm really out of bandwith to work on your issue for now... And i know it can take months before i found time to work on it !

Thx in advance :)

@goatherd
Copy link
Author

goatherd commented Jun 28, 2016

Hi Nicolas,

only dumping notices for now testing a new deployment. Having those notes often helps me determine issues faster so maybe someone else will profit from it until a fix is demanded/ provided.

As there still are several weird issues with Ubuntu 16.04. and some discussion with the MySQL 5.7 incompatibility it is not to certain that I will rely on the cookbook for now. If the team chooses MariaDB I'll provide a pull request afterwards.

Somewhat short on time as well.

@PowerKiKi
Copy link

Also trying to converge on Ubuntu 16.04 and unfortunately stucked with a repository key that is apparently missing. Without that key, packages are not installed. But the following command allow to continue:

sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys  F1656F24C74CD1D8

The thing is I don't understand where this key is coming from. I thought the mariadb repository key was CBCB082A1BB943DB ?

@goatherd did you not encounter such error ?

@goatherd
Copy link
Author

@PowerKiKi the only issue with the keyserver I know of is proxy-related: some old VM installs rejected the keyserver request. Not sure that was a host routing issue or port forwarding related.
Never had any such issues with server deployments nor with MariaDB.

Not being able to update or install from the base repository is rather weird. Are you sure the network is working and unrestricted?

Also unrelated to the issue.

@damm
Copy link

damm commented Sep 15, 2016

I didn't really have much of a problem with this cookbook on 16.04 but I ran into the issue with 10.0 being shipped; just made my wrapper call the repository to be added and .. it worked fairly painlessly

@PowerKiKi
Copy link

Thank you all for your feedback and sorry for hijacking this thread. I'll give another try in the coming weeks and open a separate issue if I can better pinpoint what's going on.

@damm
Copy link

damm commented Sep 15, 2016

@PowerKiKi no problem I had to modify the cookbook to handle the different apt key. We seem to be using a hardcoded key :(

diff --git a/cookbooks/mariadb/attributes/apt.rb b/cookbooks/mariadb/attributes/apt.rb
new file mode 100644
index 0000000..2b85044
--- /dev/null
+++ b/cookbooks/mariadb/attributes/apt.rb
@@ -0,0 +1,3 @@
+node.default['mariadb']['apt']['keyserver']="hkp://keyserver.ubuntu.com:80"
+node.default['mariadb']['apt']['key']="F1656F24C74CD1D8"
+
diff --git a/cookbooks/mariadb/recipes/repository.rb b/cookbooks/mariadb/recipes/repository.rb
index e8081eb..25ffe4b 100644
--- a/cookbooks/mariadb/recipes/repository.rb
+++ b/cookbooks/mariadb/recipes/repository.rb
@@ -15,8 +15,8 @@ if node['mariadb']['use_default_repository']
         node['mariadb']['install']['version'] + '/' + node['platform']
       distribution node['lsb']['codename']
       components ['main']
-      keyserver 'hkp://keyserver.ubuntu.com:80'
-      key '0xcbcb082a1bb943db'
+      keyserver node['mariadb']['apt']['keyserver']
+      key node['mariadb']['apt']['key']
     end
   when 'yum'
     include_recipe 'yum::default'

@joraff
Copy link

joraff commented Dec 8, 2016

Doesn't look like the above made it into a PR. May I?

@damm
Copy link

damm commented Dec 9, 2016

@joraff if it's not already done feel free.

I believe the key is different for each distribution so ideally if we are to deliver defaults; it would need to have some platform_for_values or some magic to give the right key to find per distro.

@liamdawson
Copy link

liamdawson commented Mar 2, 2017

I had to mess around with keys to get an installation happening, was having issues with this cookbook as a transitive dependency. I think the winner was this one:

sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys F1656F24C74CD1D8

but I haven't confirmed yet.

@PowerKiKi
Copy link

I never could pinpoint exactly what was going on since last September, but my solution also involved using the key F1656F24C74CD1D8 for a manual install via apt_repository resource. This was the easiest I could come up with without messing too much with this cookbook. In the end that seems to be the same as @liamdawson.

@damm
Copy link

damm commented Mar 4, 2017

The key changes based on what distro/version of distro you run. :( If that helps

shoekstra added a commit that referenced this issue Mar 5, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
shoekstra added a commit that referenced this issue Mar 5, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
shoekstra added a commit that referenced this issue Mar 5, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
shoekstra added a commit that referenced this issue Mar 5, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
@shoekstra shoekstra added this to the v1.1.0 milestone Mar 5, 2017
@shoekstra shoekstra self-assigned this Mar 5, 2017
shoekstra added a commit that referenced this issue Mar 10, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
shoekstra added a commit that referenced this issue Mar 11, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
shoekstra added a commit that referenced this issue Mar 11, 2017
- Add new APT signing key (also fixes #107)
- Add Docker tests for Travis
- Add Supermarket version badge to README
- Change VirtualBox box names to use recent versions
- Change Travis badge in README to show master build status
- Remove Fedora tests, not something we'll test going forward
@lock
Copy link

lock bot commented Jul 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants