Thanks for the Django tips, I didn't know how we could tests a template but I'll take a look at it and use a small template just for this alert then.
On Thu, Nov 20, 2014 at 8:23 PM, Sheila Miguez <email address hidden> wrote:
> I added a diff command about using "include" as an option. > > Diff comments: > > > === modified file 'capomastro/local_settings.py' > > --- capomastro/local_settings.py 2014-11-03 19:06:32 +0000 > > +++ capomastro/local_settings.py 2014-11-20 20:20:15 +0000 > > @@ -1,6 +1,5 @@ > > # SECURITY WARNING: don't run with debug turned on in production! > > -DEBUG = False > > - > > +DEBUG = True > > TEMPLATE_DEBUG = True > > ALLOWED_HOSTS = [] > > > > > > === modified file 'debian/changelog' > > --- debian/changelog 2014-11-03 19:06:32 +0000 > > +++ debian/changelog 2014-11-20 20:20:15 +0000 > > @@ -1,3 +1,10 @@ > > +capomastro (20141120) trusty; urgency=high > > + > > + * Adds warning to avoid overexposing the Jenkins server. > > + * Adds two testing scripts to use with LXC and Vagrant. > > + > > + -- Caio Begotti <email address hidden> Thu, 20 Nov 2014 > 10:41:23 -0200 > > + > > capomastro (20141103) trusty; urgency=high > > > > * Partially fixes https://bugs.launchpad.net/capomastro/+bug/1388936 > > > > === modified file 'projects/templates/projects/dependency_detail.html' > > --- projects/templates/projects/dependency_detail.html 2014-07-18 > 12:07:38 +0000 > > +++ projects/templates/projects/dependency_detail.html 2014-11-20 > 20:20:15 +0000 > > @@ -11,6 +11,9 @@ > > <div class="container"> > > <div class="row"> > > <div> > > + {% if not archiver %} > > + {% bootstrap_alert "Could not find a default archiver in the > configuration. <a href='/admin/archives/archive/'>Please set up one</a>, > otherwise artifacts of this dependency will only be available through an > exposed Jenkins server." alert_type='warning' %} > > + {% endif %} > > One way to handle it is to use an include. > > {% include "archiver_alert.html" %} > > you can also pass objects to it in case you want to tweak the snippet, for > example > > {% include "archiver_alert.html" with somevar="foobar"%} > > > <h2>{{ dependency.name }}</h2> > > {% if dependency.get_build_parameters %} > > <table class="table table-striped"> > > > > === modified file 'projects/templates/projects/dependency_form.html' > > --- projects/templates/projects/dependency_form.html 2014-06-02 > 10:55:15 +0000 > > +++ projects/templates/projects/dependency_form.html 2014-11-20 > 20:20:15 +0000 > > @@ -8,6 +8,9 @@ > > <div class="container"> > > <div class="row"> > > <div> > > + {% if not archiver %} > > + {% bootstrap_alert "Could not find a default archiver in the > configuration. <a href='/admin/archives/archive/'>Please set up one</a>, > otherwise artifacts of this dependency will only be available through an > exposed Jenkins server." alert_type='warning' %} > > + {% endif %} > > <h2>Dependency</h2> > > <form action="" method="post" class="form" id="dependency"> > > {% csrf_token %} > > > > === modified file 'projects/templates/projects/project_detail.html' > > --- projects/templates/projects/project_detail.html 2014-07-25 > 11:37:42 +0000 > > +++ projects/templates/projects/project_detail.html 2014-11-20 > 20:20:15 +0000 > > @@ -8,6 +8,9 @@ > > <div class="container"> > > <div class="row"> > > <div> > > + {% if not archiver %} > > + {% bootstrap_alert "Could not find a default archiver in the > configuration. <a href='/admin/archives/archive/'>Please set up one</a>, > otherwise artifacts of this project will only be available through an > exposed Jenkins server." alert_type='warning' %} > > + {% endif %} > > <h2>{{ project.name }}</h2> > > </div> > > </div> > > > > === modified file 'projects/templates/projects/project_form.html' > > --- projects/templates/projects/project_form.html 2014-03-20 > 16:17:58 +0000 > > +++ projects/templates/projects/project_form.html 2014-11-20 > 20:20:15 +0000 > > @@ -8,6 +8,9 @@ > > <div class="container"> > > <div class="row"> > > <div> > > + {% if not archiver %} > > + {% bootstrap_alert "Could not find a default archiver in the > configuration. <a href='/admin/archives/archive/'>Please set up one</a>, > otherwise artifacts of this project will only be available through an > exposed Jenkins server." alert_type='warning' %} > > + {% endif %} > > <h2>Project</h2> > > <form action="" method="post" class="form" id="project"> > > {% csrf_token %} > > > > === modified file 'projects/templates/projects/projectbuild_detail.html' > > --- projects/templates/projects/projectbuild_detail.html 2014-07-25 > 11:37:42 +0000 > > +++ projects/templates/projects/projectbuild_detail.html 2014-11-20 > 20:20:15 +0000 > > @@ -31,7 +31,7 @@ > > <td>{{ dependency.build.number }}</td> > > <td>{{ dependency.dependency.name }}</td> > > <td>{{ dependency.build.status }}</td> > > - <td><a href="{{ dependency.build.url }}">{{ > dependency.build.phase }}</a></td> > > + <td>{{ dependency.build.phase }}</td> > > {% else %} > > <td colspan="4">Waiting for build of job {{ dependency.job > }}</td> > > {% endif %} > > > > === modified file 'projects/views.py' > > --- projects/views.py 2014-07-25 14:58:58 +0000 > > +++ projects/views.py 2014-11-20 20:20:15 +0000 > > @@ -46,6 +46,19 @@ > > initial["auto_track"] = True > > return initial > > > > + def get_context_data(self, **kwargs): > > + """ > > + Supplement the project view with an archive check > > + """ > > + context = super( > > + ProjectCreateView, self).get_context_data(**kwargs) > > + > > + archive = get_default_archive() > > + if archive: > > + context["archiver"] = archive > > + > > + return context > > + > > > > class ProjectListView(LoginRequiredMixin, ListView): > > > > @@ -194,6 +207,11 @@ > > for artifact in > current_projectbuild.get_current_artifacts(): > > items.append(self.item_from_artifact(artifact)) > > context["current_artifacts"] = items > > + > > + archive = get_default_archive() > > + if archive: > > + context["archiver"] = archive > > + > > return context > > > > @staticmethod > > @@ -237,6 +255,20 @@ > > url_args = {"pk": self.object.pk} > > return reverse("dependency_detail", kwargs=url_args) > > > > + def get_context_data(self, **kwargs): > > + """ > > + Supplement the dependency view with an archive check > > + """ > > + context = super( > > + DependencyCreateView, self).get_context_data(**kwargs) > > + > > + archive = get_default_archive() > > + if archive: > > + context["archiver"] = archive > > + > > + return context > > + > > + > > > > class DependencyListView(LoginRequiredMixin, ListView): > > > > @@ -285,6 +317,11 @@ > > messages.add_message( > > self.request, messages.INFO, > > "Dependency currently building") > > + > > + archive = get_default_archive() > > + if archive: > > + context["archiver"] = archive > > + > > return context > > > > def post(self, request, pk): > > > > > -- > > https://code.launchpad.net/~ce-infrastructure/capomastro/capomastro-jenkins-exposure-1155/+merge/242412 > Your team CE Infrastructure is subscribed to branch lp:capomastro. >
« Back to merge proposal
Thanks for the Django tips, I didn't know how we could tests a template but
I'll take a look at it and use a small template just for this alert then.
On Thu, Nov 20, 2014 at 8:23 PM, Sheila Miguez <email address hidden>
wrote:
> I added a diff command about using "include" as an option. local_settings. py' local_settings. py 2014-11-03 19:06:32 +0000 local_settings. py 2014-11-20 20:20:15 +0000 /bugs.launchpad .net/capomastro /+bug/1388936 templates/ projects/ dependency_ detail. html' templates/ projects/ dependency_ detail. html 2014-07-18 templates/ projects/ dependency_ detail. html 2014-11-20 admin/archives/ archive/ '>Please set up one</a>, 'warning' %} alert.html" %} alert.html" with somevar="foobar"%} get_build_ parameters %} templates/ projects/ dependency_ form.html' templates/ projects/ dependency_ form.html 2014-06-02 templates/ projects/ dependency_ form.html 2014-11-20 admin/archives/ archive/ '>Please set up one</a>, 'warning' %} templates/ projects/ project_ detail. html' templates/ projects/ project_ detail. html 2014-07-25 templates/ projects/ project_ detail. html 2014-11-20 admin/archives/ archive/ '>Please set up one</a>, 'warning' %} templates/ projects/ project_ form.html' templates/ projects/ project_ form.html 2014-03-20 templates/ projects/ project_ form.html 2014-11-20 admin/archives/ archive/ '>Please set up one</a>, 'warning' %} templates/ projects/ projectbuild_ detail. html' templates/ projects/ projectbuild_ detail. html 2014-07-25 templates/ projects/ projectbuild_ detail. html 2014-11-20 build.number }}</td> dependency. name }}</td> build.status }}</td> build.url }}">{{ build.phase }}</a></td> build.phase }}</td> "auto_track" ] = True data(self, **kwargs): context_ data(** kwargs) archive( ) (LoginRequiredM ixin, ListView): projectbuild. get_current_ artifacts( ): self.item_ from_artifact( artifact) ) "current_ artifacts" ] = items archive( ) "dependency_ detail" , kwargs=url_args) data(self, **kwargs): eView, self).get_ context_ data(** kwargs) archive( ) iew(LoginRequir edMixin, ListView): add_message( archive( ) /code.launchpad .net/~ce- infrastructure/ capomastro/ capomastro- jenkins- exposure- 1155/+merge/ 242412
>
> Diff comments:
>
> > === modified file 'capomastro/
> > --- capomastro/
> > +++ capomastro/
> > @@ -1,6 +1,5 @@
> > # SECURITY WARNING: don't run with debug turned on in production!
> > -DEBUG = False
> > -
> > +DEBUG = True
> > TEMPLATE_DEBUG = True
> > ALLOWED_HOSTS = []
> >
> >
> > === modified file 'debian/changelog'
> > --- debian/changelog 2014-11-03 19:06:32 +0000
> > +++ debian/changelog 2014-11-20 20:20:15 +0000
> > @@ -1,3 +1,10 @@
> > +capomastro (20141120) trusty; urgency=high
> > +
> > + * Adds warning to avoid overexposing the Jenkins server.
> > + * Adds two testing scripts to use with LXC and Vagrant.
> > +
> > + -- Caio Begotti <email address hidden> Thu, 20 Nov 2014
> 10:41:23 -0200
> > +
> > capomastro (20141103) trusty; urgency=high
> >
> > * Partially fixes https:/
> >
> > === modified file 'projects/
> > --- projects/
> 12:07:38 +0000
> > +++ projects/
> 20:20:15 +0000
> > @@ -11,6 +11,9 @@
> > <div class="container">
> > <div class="row">
> > <div>
> > + {% if not archiver %}
> > + {% bootstrap_alert "Could not find a default archiver in the
> configuration. <a href='/
> otherwise artifacts of this dependency will only be available through an
> exposed Jenkins server." alert_type=
> > + {% endif %}
>
> One way to handle it is to use an include.
>
> {% include "archiver_
>
> you can also pass objects to it in case you want to tweak the snippet, for
> example
>
> {% include "archiver_
>
> > <h2>{{ dependency.name }}</h2>
> > {% if dependency.
> > <table class="table table-striped">
> >
> > === modified file 'projects/
> > --- projects/
> 10:55:15 +0000
> > +++ projects/
> 20:20:15 +0000
> > @@ -8,6 +8,9 @@
> > <div class="container">
> > <div class="row">
> > <div>
> > + {% if not archiver %}
> > + {% bootstrap_alert "Could not find a default archiver in the
> configuration. <a href='/
> otherwise artifacts of this dependency will only be available through an
> exposed Jenkins server." alert_type=
> > + {% endif %}
> > <h2>Dependency</h2>
> > <form action="" method="post" class="form" id="dependency">
> > {% csrf_token %}
> >
> > === modified file 'projects/
> > --- projects/
> 11:37:42 +0000
> > +++ projects/
> 20:20:15 +0000
> > @@ -8,6 +8,9 @@
> > <div class="container">
> > <div class="row">
> > <div>
> > + {% if not archiver %}
> > + {% bootstrap_alert "Could not find a default archiver in the
> configuration. <a href='/
> otherwise artifacts of this project will only be available through an
> exposed Jenkins server." alert_type=
> > + {% endif %}
> > <h2>{{ project.name }}</h2>
> > </div>
> > </div>
> >
> > === modified file 'projects/
> > --- projects/
> 16:17:58 +0000
> > +++ projects/
> 20:20:15 +0000
> > @@ -8,6 +8,9 @@
> > <div class="container">
> > <div class="row">
> > <div>
> > + {% if not archiver %}
> > + {% bootstrap_alert "Could not find a default archiver in the
> configuration. <a href='/
> otherwise artifacts of this project will only be available through an
> exposed Jenkins server." alert_type=
> > + {% endif %}
> > <h2>Project</h2>
> > <form action="" method="post" class="form" id="project">
> > {% csrf_token %}
> >
> > === modified file 'projects/
> > --- projects/
> 11:37:42 +0000
> > +++ projects/
> 20:20:15 +0000
> > @@ -31,7 +31,7 @@
> > <td>{{ dependency.
> > <td>{{ dependency.
> > <td>{{ dependency.
> > - <td><a href="{{ dependency.
> dependency.
> > + <td>{{ dependency.
> > {% else %}
> > <td colspan="4">Waiting for build of job {{ dependency.job
> }}</td>
> > {% endif %}
> >
> > === modified file 'projects/views.py'
> > --- projects/views.py 2014-07-25 14:58:58 +0000
> > +++ projects/views.py 2014-11-20 20:20:15 +0000
> > @@ -46,6 +46,19 @@
> > initial[
> > return initial
> >
> > + def get_context_
> > + """
> > + Supplement the project view with an archive check
> > + """
> > + context = super(
> > + ProjectCreateView, self).get_
> > +
> > + archive = get_default_
> > + if archive:
> > + context["archiver"] = archive
> > +
> > + return context
> > +
> >
> > class ProjectListView
> >
> > @@ -194,6 +207,11 @@
> > for artifact in
> current_
> > items.append(
> > context[
> > +
> > + archive = get_default_
> > + if archive:
> > + context["archiver"] = archive
> > +
> > return context
> >
> > @staticmethod
> > @@ -237,6 +255,20 @@
> > url_args = {"pk": self.object.pk}
> > return reverse(
> >
> > + def get_context_
> > + """
> > + Supplement the dependency view with an archive check
> > + """
> > + context = super(
> > + DependencyCreat
> > +
> > + archive = get_default_
> > + if archive:
> > + context["archiver"] = archive
> > +
> > + return context
> > +
> > +
> >
> > class DependencyListV
> >
> > @@ -285,6 +317,11 @@
> > messages.
> > self.request, messages.INFO,
> > "Dependency currently building")
> > +
> > + archive = get_default_
> > + if archive:
> > + context["archiver"] = archive
> > +
> > return context
> >
> > def post(self, request, pk):
> >
>
>
> --
>
> https:/
> Your team CE Infrastructure is subscribed to branch lp:capomastro.
>