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

Fedora 24 #3936

Closed
wants to merge 18 commits into from

Conversation

Projects
None yet
2 participants
@mvollmer
Copy link
Member

commented Mar 7, 2016

No description provided.

@mvollmer mvollmer referenced this pull request Mar 7, 2016

Closed

Testing for Fedora 24 #3912

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 7, 2016

Building and unit tests should work now.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from e4e00aa to e81e309 Mar 8, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 8, 2016

New image has been uploaded.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from e81e309 to af071c4 Mar 8, 2016

@stefwalter

This comment has been minimized.

Copy link
Contributor

commented Mar 8, 2016

Needs a rebase.

@stefwalter stefwalter added the needswork label Mar 8, 2016

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from af071c4 to 529c789 Mar 8, 2016

@mvollmer mvollmer removed the needswork label Mar 8, 2016

@stefwalter stefwalter added the priority label Mar 9, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 9, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 9, 2016

check-docker, check-multi-machine-key, and check-connection are fixed, waiting for image upload.
check-kubernetes, check-openshift, and check-system-info might be real regressions.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from 8384c62 to 2a0b662 Mar 9, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 9, 2016

check-networking hangs.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from 2a0b662 to deebc7d Mar 9, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 9, 2016

check-networking hangs.

Fixed, but it still fails.

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 9, 2016

@stefwalter

This comment has been minimized.

Copy link
Contributor

commented Mar 10, 2016

Two more commits for this branch 21ac5b2 618b0a6

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 10, 2016

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch 2 times, most recently from e9acca0 to c8f74b6 Mar 10, 2016

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 10, 2016

There is a regression in NetworkManager that causes all kinds of confusion with Cockpit (and with itself): https://bugzilla.redhat.com/show_bug.cgi?id=1316488

I'll make known issues for that.

@mvollmer

This comment has been minimized.

Copy link
Member Author

commented Mar 11, 2016

I'll make known issues for that.

Hmm, now it passes....

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from c8f74b6 to 18f9f98 Mar 11, 2016

stefwalter and others added some commits Mar 3, 2016

test: Remove fedora-22 images and add fedora-24
Fedora 24 has branched and we should be treating Fedora 23
is rather stable now ... starting to do more testing against
Fedora 24

The Atomic and Testing images still track Fedora 23 for now

Closes #3912
test: Fedora 24 image without docker or kubernetes
Neither Docker or Kubernetes are installable so just
don't install them for now. Lots of fails.
users: Adapt unit tests to newer ssh outputs
Newer versions output "no comment" instead of the filename.
test: Install docker and kubernetes in fedora-24
Plus various other related things.

Also, relax package installation.  The repositories don't seem to be
well behaved enough yet.
test: Avoid wget
It is no longer installed by default in Fedora 24 Server, and curl
does the job just fine.

mvollmer and others added some commits Mar 9, 2016

test: Don't abbreviate nmcli invocations
This protects us from changes to changes in their meaning.  For
example, "nmcli c m" means "nmcli connection monitor" in newer
versions of NetworkManager instead of "nmcli connection modify".
test: Remove etcd port changing hack
This is no longer needed, and has been fixed in kubernetes packaging.
test: Fix check-networking for NetworkManager 1.2
A connection that doesn't match any interface at the time of its
creation will never be associated with an interface, even if a
matching one shows up later.

So we first add the interface and then create the connections.

XXX - this is a workaround
test: Make less assumptions about network bonds and bridges
They might connect automatically, or they might not.  They might be
able to get an IP, or they might not be able.

This state is unsatisfying, and we can hopefully make this more
predictable in the future.
test: Really check for bond slaves
By looking for them in the right place.  Otherwise, if the slaves were
not correctly identified, the test was able to find them anyway inside
#network-interfaces.
test: Convert journal to text on the machine itself
Older versions of journalctl can't read the files produced by newer
versions of journald.

Also, don't download the raw journal files.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from 18f9f98 to 3e5ec04 Mar 11, 2016

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from 3e5ec04 to 2255279 Mar 11, 2016

test Merge check-registry into check-openshift
Because they both use the singleton 'openshift' machine.

@mvollmer mvollmer force-pushed the mvollmer:fedora-24 branch from 83f69e7 to ea95953 Mar 11, 2016

stefwalter added a commit that referenced this pull request Mar 11, 2016

test: Add known issue files for kubernetes regressions on fedora-24
Closes #3936
Reviewed-by: Stef Walter <stefw@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.