Re: [Spacewalk-devel] Future of the development stack?

2013-10-15 Thread Duncan Mac-Vicar P.
On 14/10/13 11:37, Cliff Perry wrote:
 Hi Duncan and guys,
 so the last I saw, 12 days ago, was this Spacewalk branch
 
 https://git.fedorahosted.org/cgit/spacewalk.git/log/?h=bootstrap-css
 
 Grant loaded it onto a test system and it was looking good, still some
 polish needed and fixes done in a few places, but great to see.

Hey Cliff

Thanks for the feedback, it is much appreciated.

 So, I had a couple of questions
 
  1) Any further progress/merging into that branch?

I spent the Hackweek at SUSE porting the list widgets. Grant Gainey
supported me with useful information. Silvio just pushed the current
branch to the spacewalk git.

The current plan is to make a sprint this and next week and look into:

- Perl pages (we upstreamed some Java ports, but as part of a different
feature branch :-))
- Details everywhere
- asset processing and CDNs (I will explain below)

Then we would like to try to merge upstream, and work the next details
(static html pages that require different classes, by-hand widgets and
porting images to font icons that are not generated by jsp tags) in a
second round when the first basic stuff is already merged. This would
open the window for others to contribute, port, and to get some bug reports.

We would continue then to do minor work as we find issues and in a ~6
weeks we will do another 2 week sprint focusing various developers on
polishing specific reported problems.

So our main focus now is to manage to get this week and next week
everything in a state where a merge would be accepted.

  2) There was at least two css scripts being pulled from 3rd party
 locations which were required. They need to be packaged/shipped - since
 a fully disconnected Spacewalk instance, would not have internet access
 to pull it down.
 

Yes, this is the asset processing part I mention above. We did this for
development convenience.
We will remove this and use bundled versions.
Also, right now LESS is preprocessed to CSS in the client using less.js.
We want to introduce a developer/production mode that in the second case
it uses the css (generated during build) and in the first it would
preprocess at runtime for developer convenience. Also this processing
could include minifying and bundling of various assets together. This
could help eg. RHN by reducing bandwidth and number of requests. Rails
has this feature, may be we can mimick it somehow with Struts.

  3) More a reminder, to ensure all code/libraries/scripts used are GPL
 friendly.
 http://www.gnu.org/licenses/license-list.html#GPLCompatibleLicenses

Twitter Bootstrap: Apache License 2.0
jQuery: MIT
Font-Awesome code: MIT
(all compatible)
Font-Awesome fontss: SIL Open Font License (OFL)
Addressed here: http://www.gnu.org/licenses/license-list.html#SILOFL
(not a problem)

 Really, this is good stuff and I look forward to seeing the finished and
 polished product.

Thanks a lot for the feedback!

-- 
Duncan Mac-Vicar P. - http://www.suse.com/

SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix
Imendörffer, HRB 16746 (AG Nürnberg)
Maxfeldstraße 5, 90409 Nürnberg, Germany

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


[Spacewalk-devel] [PATCH] python pgsql tests: made easier Jenkins integration

2013-10-15 Thread Flavio Castelli
Made easier to run the pgsql tests using nosetests in conjunction with
the 'with-xunit'.

Invoking:
  nosetests --with-xunit --xunit-file pgsql_tests.xml runtests-postgresql.py

will produce an xml file which can be parsed by Jenkins.

Invoking the script in the old way still works as expected.
---
 backend/test/non-unit/server/rhnSQL/dbtests.py | 45 --
 backend/test/runtests-postgresql.py| 42 ++--
 2 files changed, 39 insertions(+), 48 deletions(-)

diff --git a/backend/test/non-unit/server/rhnSQL/dbtests.py 
b/backend/test/non-unit/server/rhnSQL/dbtests.py
index ff51148..0fa703a 100644
--- a/backend/test/non-unit/server/rhnSQL/dbtests.py
+++ b/backend/test/non-unit/server/rhnSQL/dbtests.py
@@ -201,44 +201,6 @@ class RhnSQLDatabaseTests(unittest.TestCase):
 
 
 
-class PostgreSQLDatabaseTests(RhnSQLDatabaseTests):
-QUERY_CREATE_TABLE = 
-CREATE TABLE %s(id INT, name TEXT, num NUMERIC(5,2))
-
-
-SIMPLE_PROCEDURE = 
-CREATE OR REPLACE FUNCTION return_int(returnme INTEGER) RETURNS int AS $$
-DECLARE
-myInt int;
-BEGIN
-myInt := returnme;
-RETURN myInt;
-END
-$$ LANGUAGE 'plpgsql';
-
-
-def setUp(self):
-self.set_temp_table()
-create_table_query = self.QUERY_CREATE_TABLE % self.temp_table
-cursor = rhnSQL.prepare(create_table_query)
-cursor.execute()
-
-RhnSQLDatabaseTests.setUp(self)
-
-cursor = rhnSQL.prepare(self.SIMPLE_PROCEDURE)
-cursor.execute()
-
-def tearDown(self):
-try:
-cursor = rhnSQL.prepare(DROP FUNCTION return_int(returnme 
integer))
-cursor.execute()
-except:
-pass
-
-RhnSQLDatabaseTests.tearDown(self)
-
-
-
 class OracleDatabaseTests(RhnSQLDatabaseTests):
 QUERY_CREATE_TABLE = 
 CREATE TABLE %s(id NUMBER, name VARCHAR2(256), num NUMBER(5,2))
@@ -270,13 +232,6 @@ END;
 
 RhnSQLDatabaseTests.tearDown(self)
 
-
-
-def postgresql_suite():
-suite = unittest.TestSuite()
-suite.addTest(unittest.makeSuite(PostgreSQLDatabaseTests))
-return suite
-
 def oracle_suite():
 suite = unittest.TestSuite()
 suite.addTest(unittest.makeSuite(OracleDatabaseTests))
diff --git a/backend/test/runtests-postgresql.py 
b/backend/test/runtests-postgresql.py
index 0a10ec7..f647331 100755
--- a/backend/test/runtests-postgresql.py
+++ b/backend/test/runtests-postgresql.py
@@ -48,11 +48,47 @@ rhnSQL.initDB(backend=postgresql, host=PG_HOST, 
username=PG_USER,
 rhnSQL.initDB(backend=postgresql, host=PG_HOST, username=PG_USER,
 password=PG_PASSWORD, database=PG_DATABASE)
 
+class PostgreSQLDatabaseTests(dbtests.RhnSQLDatabaseTests):
+QUERY_CREATE_TABLE = 
+CREATE TABLE %s(id INT, name TEXT, num NUMERIC(5,2))
+
+
+SIMPLE_PROCEDURE = 
+CREATE OR REPLACE FUNCTION return_int(returnme INTEGER) RETURNS int AS $$
+DECLARE
+myInt int;
+BEGIN
+myInt := returnme;
+RETURN myInt;
+END
+$$ LANGUAGE 'plpgsql';
+
+
+def setUp(self):
+self.set_temp_table()
+create_table_query = self.QUERY_CREATE_TABLE % self.temp_table
+cursor = rhnSQL.prepare(create_table_query)
+cursor.execute()
+
+dbtests.RhnSQLDatabaseTests.setUp(self)
+
+cursor = rhnSQL.prepare(self.SIMPLE_PROCEDURE)
+cursor.execute()
+
+def tearDown(self):
+try:
+cursor = rhnSQL.prepare(DROP FUNCTION return_int(returnme 
integer))
+cursor.execute()
+except:
+pass
+
+dbtests.RhnSQLDatabaseTests.tearDown(self)
+
 def suite():
+s = unittest.TestSuite()
+s.addTest(unittest.makeSuite(PostgreSQLDatabaseTests))
 # Append all test suites here:
-return unittest.TestSuite((
-dbtests.postgresql_suite(),
-   ))
+return unittest.TestSuite(s)
 
 if __name__ == __main__:
 try:
-- 
1.8.1.4

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


Re: [Spacewalk-devel] [python pgsql tests] make easier to integrate them with Jenkins

2013-10-15 Thread Michael Mraka
Flavio Castelli wrote:
% This commit allows to invoke the python test suite interacting with the 
PostgreSQL
% straight from the cli using nosetests.
% In this way it's possible to use the xunit features provided by nosetests and
% plug the test results into Jenkins.
% As you can see the changes are trivial.

Hello Flavio,

I've commited your patch.
Thanks.


--
Michael Mráka
Satellite Engineering, Red Hat

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel