Projects tigase _server server-core Issues #649
Installer issue (#649)
Andrzej Wójcik (Tigase) opened 9 years ago
Due Date
2016-03-03

Installer of Tigase XMPP Server 7.0.2 fails to copy file which makes it impossible to use HTTP API when Tigase XMPP Server is installer using installer.

Andrzej Wójcik (Tigase) commented 9 years ago

I fixed this issue.

Andrzej Wójcik (Tigase) commented 9 years ago

I do not know how serious this is, but I see that we have 2 fixes for 7.0.3 waiting in repository, so we can release this as 7.0.3 or wait for release of 7.1.0.

Artur Hefczyc commented 9 years ago

Let's release 7.0.3 asap. Maybe at the end of the month? I would like Daniel to check the installer and all the other stuff.

Daniel Wisnewski commented 9 years ago

Installer does not install required http component files when selected. In fact, no verbose error shows as a result, log file only reports:

2016-02-29 07:46:11.151 [main]             MessageRouterConfig.hasClassForServerComponent()  WARNING: Can't load component rest: tigase.http.rest.RestMessageReceiver

and otherwise operates as normal.

Wrapper install is also broken, as config file references old /lib file, this too should be fixed for release.


C:\Tigase\jars>dir
 Volume in drive C has no label.
 Volume Serial Number is 0C75-26AB

 Directory of C:\Tigase\jars

02/29/2016  07:32 AM    <DIR>          .
02/29/2016  07:32 AM    <DIR>          ..
05/14/2015  10:55 AM         2,703,892 derby.jar
05/14/2015  10:55 AM           174,932 derbytools.jar
05/14/2015  10:55 AM         6,317,783 groovy-all.jar
05/14/2015  02:44 PM            45,024 hamcrest-core.jar
05/14/2015  02:44 PM            68,274 jstun.jar
05/14/2015  02:44 PM           301,946 jtds.jar
05/14/2015  02:44 PM            22,922 licence-lib.jar
05/14/2015  02:44 PM           590,996 mongo-java-driver.jar
05/14/2015  10:55 AM           703,265 mysql-connector-java.jar
05/14/2015  10:55 AM           551,290 postgresql.jar
05/14/2015  02:44 PM            67,322 tigase-acs-muc.jar
05/14/2015  02:44 PM            33,408 tigase-acs-pubsub.jar
05/14/2015  02:44 PM            56,815 tigase-acs.jar
05/14/2015  02:44 PM            43,441 tigase-extras.jar
05/14/2015  02:44 PM           209,250 tigase-http-api.jar
05/14/2015  02:44 PM            55,141 tigase-message-archiving.jar
05/14/2015  02:44 PM            57,476 tigase-mongodb.jar
05/14/2015  02:44 PM           157,257 tigase-muc.jar
05/14/2015  02:44 PM           274,716 tigase-pubsub-2.2.0.jar.old
05/14/2015  02:44 PM           354,973 tigase-pubsub.jar
05/14/2015  02:44 PM         1,990,114 tigase-server.jar
05/14/2015  02:44 PM            60,275 tigase-socks5.jar
05/14/2015  02:44 PM             8,656 tigase-stats-collector-library.jar
05/14/2015  02:44 PM             7,689 tigase-stats-collector-provider.jar
05/14/2015  02:44 PM            12,484 tigase-stun.jar
05/14/2015  02:44 PM         6,998,410 tigase-web-ui.war
05/14/2015  10:55 AM            81,920 wrapper.dll
05/14/2015  10:55 AM            83,820 wrapper.jar
              28 File(s)     22,033,491 bytes
               2 Dir(s)  51,261,181,952 bytes free

Windows/linux installs yield same result, same file structure.

Andrzej Wójcik (Tigase) commented 9 years ago

I checked and installer created wrong configuration file as class RestMessageReceiver was available only in version 1.0.0 of HTTP API Component and in version 1.1.0 it was replaced by HttpMessageReceiver component. I applied needed changes to instalator for version 7.0.3-SNAPSHOT.

As for jar files, classes are properly copied.

warpper.conf file is wrong however it is hard for me to tell how it should be fixed, as in new version we fixed it by update of wrapper and replacing of all classpath entries with single line:

wrapper.java.classpath.1=jars/*.jar

%daniel If you have idea how to fix wrapper then please fix it or let me know how it should be fixed.

Daniel Wisnewski commented 9 years ago

I updated wrapper.conf for stable branch after some testing. The fixes run the wrapper without errors. Please review the git to make sure I didn't break anything on other branches (I followed your workflow).

If we are placing fixes for v7.0.2, we should cherry pick the fix from #3446 for this branch as well.

Andrzej Wójcik (Tigase) commented 9 years ago

Thank you for fixing wrapper.conf.

I created new branch for HTTP API for which we needed to cherry pick fix - now in HTTP API is new branch named stable which contains fixed version of HTTP API component (with changes from #3446). This new version is marked as 1.1.1-SNAPSHOT@, so I updated @tigase-server project in stable branch to use this new version for building @7.0.3-SNAPSHOT@.

Due to that to test new version you will most likely need to download tigase-http-api@, switch to @stable branch and compile and install this version locally to allow Tigase XMPP Server 7.0.3-SNAPSHOT to resolve properly dependencies when creating distribution files and installer.

Before release of 7.0.3 we will need to release 1.1.1 version of HTTP API and update pom.xml responsible for creation of distribution packages to use this new version @1.1.1@.

%wojtek Could you confirm?

wojciech.kapcia@tigase.net commented 9 years ago

Andrzej Wójcik wrote:

Before release of 7.0.3 we will need to release 1.1.1 version of HTTP API and update pom.xml responsible for creation of distribution packages to use this new version @1.1.1@.

wojtek Could you confirm?

Yes, this is correct approach. Release of 1.1.1 should be straightforward and can be done from maven (it will automatically tag and change versions and push everything to the repository). One thing to keep in mind would be to set next version in stable branch to 1.1.2-SNAPSHOT

Daniel Wisnewski commented 9 years ago

%kobit this appears resolved. Do we have other issues for v7.0.3 we want to push?

Artur Hefczyc commented 9 years ago

Wojciech, do you have anything other which is worth including in 7.0.3 bug fix version?

wojciech.kapcia@tigase.net commented 9 years ago

I've went over the changes in master@/@release since latest stable release and I don't think there is any significant issue/commit worth backporting.

Artur Hefczyc commented 9 years ago

Ok, let's go ahead with 7.0.3 release.

%wojtek could you please start the process and assign tasks?

wojciech.kapcia@tigase.net commented 9 years ago

#3997 crated.

issue 1 of 1
Type
Bug
Priority
Normal
Assignee
RedmineID
3959
Spent time
21h 15m
Issue Votes (0)
Watchers (0)
Reference
tigase/_server/server-core#649
Please wait...
Page is in error, reload to recover