Skip to content

Commit

Permalink
Temporarily disable BIRD in test-bgp-router. #142
Browse files Browse the repository at this point in the history
Alpine updated their Bird image to 2.0, which breaks the generated
config and makes bird behave weirdly in other ways. I need to sleep
right now, so this is a quick bandaid while I figure out a real fix.
  • Loading branch information
danderson committed Jan 17, 2018
1 parent f04bf6f commit 3d3cf0a
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 6 deletions.
7 changes: 4 additions & 3 deletions manifests/tutorial-1.yaml
Expand Up @@ -6,9 +6,10 @@ metadata:
data:
config: |
peers:
- my-asn: 64512
peer-asn: 64512
peer-address: 10.96.0.100
# Temporarily disabled due to bug #142
# - my-asn: 64512
# peer-asn: 64512
# peer-address: 10.96.0.100
- my-asn: 64512
peer-asn: 64512
peer-address: 10.96.0.101
Expand Down
7 changes: 4 additions & 3 deletions test-bgp-router/bird.go
Expand Up @@ -4,14 +4,15 @@ import (
"fmt"
"io/ioutil"
"net"
"os"
"os/exec"
"strings"
)

func hasBird() bool {
_, err := os.Stat("/usr/sbin/bird")
return err == nil
// Disabled pending bug #142
// _, err := os.Stat("/usr/sbin/bird")
// return err == nil
return false
}

func writeBirdConfig() error {
Expand Down
6 changes: 6 additions & 0 deletions website/content/tutorial/minikube.md
Expand Up @@ -67,6 +67,12 @@ will be configured to speak BGP, but won't configure Linux to forward
traffic based on the data they receive. Instead, we'll just inspect
that data to see what a real router _would_ do.

{{% notice note %}}
BIRD is temporarily disabled due to
a [bug](https://github.com/google/metallb/issues/142), so screenshots
will look slightly different.
{{% /notice %}}

Deploy these test routers with `kubectl`:

`kubectl apply -f https://raw.githubusercontent.com/google/metallb/v0.3.0/manifests/test-bgp-router.yaml`
Expand Down

0 comments on commit 3d3cf0a

Please sign in to comment.