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

Officially support Ubuntu 18.04 #758

Merged
merged 1 commit into from
Jan 16, 2019

Conversation

baurmatt
Copy link
Contributor

@baurmatt baurmatt commented Jan 15, 2019

Pull Request (PR) description

This PR adds official (metadata.json + Tests) for Ubuntu 18.04.

This Pull Request (PR) fixes the following issues

Fixes #757

@baurmatt
Copy link
Contributor Author

baurmatt commented Jan 15, 2019

Hmm this is strange:

    Port "15672"

ubuntu1804-64-1 17:09:00$ /bin/sh -c ss\ -tnl4\ \|\ grep\ --\ \\\ \\\\\\\*:15672\\\

ubuntu1804-64-1 executed in 0.01 seconds
root@ubuntu1804-64-1:/# netstat -nlp | grep 15672
tcp        0      0 0.0.0.0:15672           0.0.0.0:*               LISTEN      2410/beam.smp

The problem seems to be that its trying to grep for *:15672 due to this line in specinfra https://github.com/mizzy/specinfra/blob/7fcb239eb5b4dca705663ca342c405c41a5ffc0c/lib/specinfra/command/module/ss.rb#L17

I guess the output off ss changed.

Ubuntu 18.04:

root@ubuntu1804-64-1:/# ss -tnl4 | grep 15672
LISTEN   0         128                 0.0.0.0:15672            0.0.0.0:*
root@ubuntu1804-64-1:/# which ss
/bin/ss
root@ubuntu1804-64-1:/# dpkg -S /bin/ss
iproute2: /bin/ss
root@ubuntu1804-64-1:/# dpkg -l | grep iproute2
ii  iproute2                    4.15.0-2ubuntu1                   amd64        networking and traffic control tools

Ubuntu 16.04:

root@ubuntu1604-64-1:/# ss -tnl4 | grep 15672
LISTEN     0      128          *:15672                    *:*
root@ubuntu1604-64-1:/# which ss
/bin/ss
root@ubuntu1604-64-1:/# dpkg -S /bin/ss
iproute2: /bin/ss
root@ubuntu1604-64-1:/# dpkg -l | grep iproute2
ii  iproute2                   4.3.0-1ubuntu3.16.04.4                amd64        networking and traffic control tools

@baurmatt
Copy link
Contributor Author

@wyardley
Copy link
Contributor

Good find. It would be better if we can avoid just marking the tests of the service listening as pending — want to try and get this fixed upstream?

@baurmatt
Copy link
Contributor Author

@wyardley Just opened a issue for that ;) Take a look, perhaps you have a good idea of how we can fix this :)

@baurmatt baurmatt closed this Jan 16, 2019
@baurmatt baurmatt reopened this Jan 16, 2019
@baurmatt baurmatt closed this Jan 16, 2019
@baurmatt baurmatt reopened this Jan 16, 2019
@wyardley
Copy link
Contributor

wyardley commented Jan 16, 2019

@baurmatt I can restart Travis, but not sure this will work until the bundle gets updated with newer serverspec version (which may have to come from https://github.com/voxpupuli/modulesync_config)

Opening and closing won't re-run Travis, but force-pushing will.

@baurmatt
Copy link
Contributor Author

Oh really? Since when is that? Worked for me a couple of weeks ago pretty well.

Travis should update the specinfra gem as we do not pin it directly.

@wyardley
Copy link
Contributor

@baurmatt Travis also seems to be having some issues right now, not running on any of the new PRs :/

You're right about specinfra; forgot that Gemfile.lock wasn't committed when I looked locally.

Also, seems like the build is passing.

@wyardley wyardley merged commit f6f7b6e into voxpupuli:master Jan 16, 2019
@wyardley wyardley added the enhancement New feature or request label Jan 17, 2019
@wyardley
Copy link
Contributor

@baurmatt: can you open a separate PR to add it to docker sets in https://github.com/voxpupuli/puppet-rabbitmq/blob/master/.sync.yml ?

@bastelfreak
Copy link
Member

FYI: The travis issues were caused by a full queue of jobs.

@baurmatt baurmatt deleted the feature/support18.04 branch January 18, 2019 09:16
@baurmatt
Copy link
Contributor Author

I keep forgetting about modulesync :( Openend up #767 to make it persistent.

@wyardley wyardley added enhancement New feature or request and removed enhancement New feature or request labels Jan 20, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-rabbitmq that referenced this pull request Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Officially support Ubuntu 18.04
3 participants