diff --git a/.gitignore b/.gitignore
index e8ff56275..7628f9af7 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,6 +1,6 @@
*.pyc
configuration.py
.idea
-*.sh
+/*.sh
fabfile.py
diff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 000000000..a7f9cda45
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,7 @@
+language: python
+python:
+ - "2.7"
+install:
+ - pip install -r requirements.txt
+script:
+ - ./scripts/cibuild.sh
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index d5e9bd38e..3c36885ea 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -48,3 +48,9 @@ Even if it's not quite right for NetBox, we may be able to point you to a tool b
* A use case for the feature; who would use it and what value it would add to NetBox
* A rough description of any changes necessary to the database schema (if applicable)
* Any third-party libraries or other resources which would be involved
+
+# Submitting Pull Requests
+
+When submitting a pull request, please be sure to work off of branch `develop`, rather than branch `master`.
+In NetBox, the `develop` branch is used for ongoing development, while `master` is used for tagging new
+stable releases.
diff --git a/README.md b/README.md
index ba736fee8..ab739f9e3 100644
--- a/README.md
+++ b/README.md
@@ -1,7 +1,20 @@
+# NetBox
+
NetBox is an IP address management (IPAM) and data center infrastructure management (DCIM) tool. Initially conceived by the network engineering team at [DigitalOcean](https://www.digitalocean.com/), NetBox was developed specifically to address the needs of network and infrastructure engineers.
NetBox runs as a web application atop the [Django](https://www.djangoproject.com/) Python framework with a [PostgreSQL](http://www.postgresql.org/) database. For a complete list of requirements, see `requirements.txt`. The code is available [on GitHub](https://github.com/digitalocean/netbox).
+Questions? Comments? Please join us on IRC in **#netbox** on **irc.freenode.net**!
+
+### Build Status
+
+| | python 2.7 |
+|-------------|------------|
+| **master** | [](https://travis-ci.org/digitalocean/netbox) |
+| **develop** | [](https://travis-ci.org/digitalocean/netbox) |
+
+## Screenshots
+


diff --git a/docs/getting-started.md b/docs/getting-started.md
index 8f2688aef..7395fdace 100644
--- a/docs/getting-started.md
+++ b/docs/getting-started.md
@@ -206,20 +206,26 @@ Now if we navigate to the name or IP of the server (as defined in `ALLOWED_HOSTS
If the test service does not run, or you cannot reach the NetBox home page, something has gone wrong. Do not proceed with the rest of this guide until the installation has been corrected.
-# nginx and gunicorn
+# Web Server and gunicorn
## Installation
-We'll set up a simple HTTP front end using [nginx](https://www.nginx.com/resources/wiki/) and [gunicorn](http://gunicorn.org/) for the purposes of this guide. (You are of course free to use whichever combination of HTTP and WSGI services you'd like.) We'll also use [supervisord](http://supervisord.org/) for service persistence.
+We'll set up a simple HTTP front end using [gunicorn](http://gunicorn.org/) for the purposes of this guide. For web servers, we have 2 configurations ready to go - we provide instructions for both [nginx](https://www.nginx.com/resources/wiki/)and [Apache](http://httpd.apache.org/docs/2.4). (You are of course free to use whichever combination of HTTP and WSGI services you'd like.) We'll also use [supervisord](http://supervisord.org/) for service persistence.
```
-# apt-get install nginx gunicorn supervisor
+# apt-get install gunicorn supervisor
```
## nginx Configuration
The following will serve as a minimal nginx configuration. Be sure to modify your server name and installation path appropriately.
+```
+# apt-get install nginx
+```
+
+Once nginx is installed, proceed with the following configuration:
+
```
server {
listen 80;
@@ -256,6 +262,40 @@ Restart the nginx service to use the new configuration.
# service nginx restart
* Restarting nginx nginx
```
+## Apache Configuration
+
+If you're feeling adventurous, or you already have Apache installed and can't run a dual-stack on your server - an Apache configuration has been created:
+
+```
+
+ ProxyPreserveHost On
+
+ ServerName netbox.totallycool.tld
+
+ Alias /static/ /opt/netbox/static/static
+
+
+ Options Indexes FollowSymLinks MultiViews
+ AllowOverride None
+ Order allow,deny
+ Allow from all
+ #Require all granted [UNCOMMENT THIS IF RUNNING APACHE 2.4]
+
+
+
+ ProxyPass !
+
+
+ ProxyPass / http://127.0.0.1:8001;
+ ProxyPassReverse / http://127.0.0.1:8001;
+
+```
+
+Save the contents of the above example in `/etc/apache2/sites-available/netbox.conf`, add in the newly saved configuration and reload Apache:
+
+```
+# a2ensite netbox; service apache2 restart
+```
## gunicorn Configuration
@@ -289,3 +329,4 @@ Finally, restart the supervisor service to detect and run the gunicorn service:
At this point, you should be able to connect to the nginx HTTP service at the server name or IP address you provided. If you are unable to connect, check that the nginx service is running and properly configured. If you receive a 502 (bad gateway) error, this indicates that gunicorn is misconfigured or not running.
Please keep in mind that the configurations provided here are a bare minimum to get NetBox up and running. You will almost certainly want to make some changes to better suit your production environment.
+
diff --git a/docs/ipam.md b/docs/ipam.md
index 295b32ccc..d37a16319 100644
--- a/docs/ipam.md
+++ b/docs/ipam.md
@@ -36,7 +36,7 @@ Any prefixes you create in NetBox (discussed below) will be automatically organi
Regional Internet Registries (RIRs) are responsible for the allocation of global address space. The five RIRs are ARIN, RIPE, APNIC, LACNIC, and AFRINIC. However, some address space has been set aside for private or internal use only, such as defined in RFCs 1918 and 6598. NetBox considers these RFCs as a sort of RIR as well; that is, an authority which "owns" certain address space.
-Each aggregate must be assigned to one RIR. NetBox by default will be populated with the RIRs listed above, however you are free to remove these and/or create your own if you choose.
+Each aggregate must be assigned to one RIR. You are free to define whichever RIRs you choose (or create your own).
---
diff --git a/netbox/dcim/models.py b/netbox/dcim/models.py
index 5ff233c9c..0c587dd3d 100644
--- a/netbox/dcim/models.py
+++ b/netbox/dcim/models.py
@@ -188,10 +188,11 @@ class Rack(CreatedUpdatedModel):
# Validate that Rack is tall enough to house the installed Devices
if self.pk:
top_device = Device.objects.filter(rack=self).order_by('-position').first()
- min_height = top_device.position + top_device.device_type.u_height - 1
- if self.u_height < min_height:
- raise ValidationError("Rack must be at least {}U tall with currently installed devices."
- .format(min_height))
+ if top_device:
+ min_height = top_device.position + top_device.device_type.u_height - 1
+ if self.u_height < min_height:
+ raise ValidationError("Rack must be at least {}U tall with currently installed devices."
+ .format(min_height))
def to_csv(self):
return ','.join([
diff --git a/netbox/ipam/views.py b/netbox/ipam/views.py
index 75c582b99..b230783fc 100644
--- a/netbox/ipam/views.py
+++ b/netbox/ipam/views.py
@@ -395,16 +395,24 @@ def ipaddress(request, pk):
ipaddress = get_object_or_404(IPAddress.objects.select_related('interface__device'), pk=pk)
+ # Parent prefixes table
parent_prefixes = Prefix.objects.filter(vrf=ipaddress.vrf, prefix__net_contains=str(ipaddress.address.ip))
- related_ips = IPAddress.objects.select_related('interface__device').exclude(pk=ipaddress.pk)\
- .filter(vrf=ipaddress.vrf, address__net_contained_or_equal=str(ipaddress.address))
+ parent_prefixes_table = tables.PrefixBriefTable(parent_prefixes)
+ # Duplicate IPs table
+ duplicate_ips = IPAddress.objects.filter(vrf=ipaddress.vrf, address=str(ipaddress.address))\
+ .exclude(pk=ipaddress.pk).select_related('interface__device', 'nat_inside')
+ duplicate_ips_table = tables.IPAddressBriefTable(duplicate_ips)
+
+ # Related IP table
+ related_ips = IPAddress.objects.select_related('interface__device').exclude(address=str(ipaddress.address))\
+ .filter(vrf=ipaddress.vrf, address__net_contained_or_equal=str(ipaddress.address))
related_ips_table = tables.IPAddressBriefTable(related_ips)
- RequestConfig(request, paginate={'klass': EnhancedPaginator}).configure(related_ips_table)
return render(request, 'ipam/ipaddress.html', {
'ipaddress': ipaddress,
- 'parent_prefixes': parent_prefixes,
+ 'parent_prefixes_table': parent_prefixes_table,
+ 'duplicate_ips_table': duplicate_ips_table,
'related_ips_table': related_ips_table,
})
diff --git a/netbox/templates/ipam/ipaddress.html b/netbox/templates/ipam/ipaddress.html
index 4c2d11f10..6834ded9e 100644
--- a/netbox/templates/ipam/ipaddress.html
+++ b/netbox/templates/ipam/ipaddress.html
@@ -119,31 +119,14 @@
-
-
- Parent Prefixes
-
- {% if parent_prefixes %}
-
- {% for p in parent_prefixes %}
-
-
- {{ p }}
- |
-
- {% if p.site %}
- {{ p.site }}
- {% endif %}
- |
- {{ p.status }} |
- {{ p.role }} |
-
- {% endfor %}
-
- {% else %}
-
None
- {% endif %}
-
+ {% with heading='Parent Prefixes' %}
+ {% render_table parent_prefixes_table 'panel_table.html' %}
+ {% endwith %}
+ {% if duplicate_ips_table.rows %}
+ {% with heading='Duplicate IP Addresses' panel_class='danger' %}
+ {% render_table duplicate_ips_table 'panel_table.html' %}
+ {% endwith %}
+ {% endif %}
{% with heading='Related IP Addresses' %}
{% render_table related_ips_table 'panel_table.html' %}
{% endwith %}
diff --git a/scripts/cibuild.sh b/scripts/cibuild.sh
new file mode 100755
index 000000000..91a847c37
--- /dev/null
+++ b/scripts/cibuild.sh
@@ -0,0 +1,28 @@
+#!/bin/bash
+
+# Exit code starts at 0 but is modified if any checks fail
+EXIT=0
+
+# Output a line prefixed with a timestamp
+info()
+{
+ echo "$(date +'%F %T') |"
+}
+
+# Track number of seconds required to run script
+START=$(date +%s)
+echo "$(info) starting build checks."
+
+# Syntax check all python source files
+SYNTAX=$(find . -name "*.py" -type f -exec python -m py_compile {} \; 2>&1)
+if [[ ! -z $SYNTAX ]]; then
+ echo -e "$SYNTAX"
+ echo -e "\n$(info) detected one or more syntax errors, failing build."
+ EXIT=1
+fi
+
+# Show build duration
+END=$(date +%s)
+echo "$(info) exiting with code $EXIT after $(($END - $START)) seconds."
+
+exit $EXIT